Commit Graph

5063 Commits

Author SHA1 Message Date
Tessa Kelly
e526f3c250 Remove accordion 2022-08-16 14:27:50 -07:00
charbelrami
5e79097a49 bump version to 17.5.0 2022-08-16 14:47:56 -03:00
Charbel
d2c181e3dd
Merge pull request #1030 from NoRedInk/selected-tabpanel-data-attr
A11-1357 add data-selected to tabpanel for testing purposes
2022-08-16 14:14:29 -03:00
charbelrami
082477c8e8 add data-selected to tabpanel for testing purposes 2022-08-16 13:53:50 -03:00
Ben Dansby
5bb1fec6ab
Merge pull request #1029 from NoRedInk/revert-1009-dansby/checkbox-bg
Revert "Remove white background from checkbox"
2022-08-15 16:47:12 -07:00
Ben Dansby
0fdf4c17c1
Revert "Remove white background from checkbox" 2022-08-15 16:36:04 -07:00
Ben Dansby
3b553c5368
Merge pull request #1027 from NoRedInk/dansby/heading-line-height
Tighten and standardize heading line heights
2022-08-11 15:29:24 -07:00
Ben Dansby
dabe01a090 change heading line heights 2022-08-11 10:03:55 -07:00
Tessa
f038618a20
Merge pull request #1025 from NoRedInk/tessa/kebab
Adds UiIcon.kebab
2022-08-10 16:14:49 -07:00
Tessa
6bca8d5314
Merge pull request #1026 from NoRedInk/growth/onboarding-icons
Add apple and briefcase icons
2022-08-10 12:49:50 -07:00
Tessa Kelly
4e669e85a7 adds kebab 2022-08-10 11:21:31 -07:00
Brian J. Cardiff
074deca0bb Add apple and briefcase icons 2022-08-10 15:18:00 -03:00
noredinkbot-duet[bot]
f91595c227
Merge #1024
1024: Resolves A11-1254, A11-1235, and A11-1277: remove roving tabindex and `aria-hidden` from tab panels, add a fixed `tabindex="0"` r=charbelrami a=charbelrami

Roving tabindex is not necessary on the panels. Panels with `display: none` are not focusable even with `tabindex="0"`. Added a fixed `tabindex="0"` to all panels to ensure they are focusable even when there are no focusable descendants.

Co-authored-by: charbelrami <charbel.f.rami@gmail.com>
2022-08-09 22:51:11 +00:00
charbelrami
da1a57e052 remove roving tabindex and aria-hidden from panels 2022-08-09 19:10:12 -03:00
Tessa
31c7d513a0
Merge pull request #1023 from NoRedInk/release/17.4.0
bump 17.4.0
2022-08-09 14:24:47 -07:00
Erik Feeley
f45cc596c5 bump 17.4.0 2022-08-09 16:34:43 -04:00
noredinkbot-duet[bot]
ad9e6c5197
Merge #1021
1021: Adds disabled and loading attributes/config r=erik-nri a=erik-nri

This implements similar attributes to how it is being done over in
TextInput.

### Motivation
For hackday I was updating the sign up with email form and adding the disabled / loading attributes for the TextInputs. I noticed that the Select did not have similar functionality and thought it might be worth adding 😄.

One odd thing to note, when adding the fields to the examples app, I noticed that I needed to flip the order in the list of the select input and its label. If I did not the styles did not work out for some reason. I was pairing with `@charbelrami` at the time and we decided to hold off on spending a bunch of time trying to figure out why this was happening.

This update only incurs a minor change!

Co-authored-by: Erik Feeley <erikfeeley@eriks-mbp.turkey-snapper.ts.net>
Co-authored-by: Erik Feeley <erik@noredink.com>
2022-08-09 20:15:52 +00:00
Erik Feeley
036e6bbbf8 dont allow disabled select to look clickable 2022-08-08 23:53:48 -04:00
noredinkbot-duet[bot]
b5b2882f82
Merge #1017
1017: A11-1234 add `type="button"` to tabs r=charbelrami a=charbelrami

adds `type="button"` to prevent form submissions

Co-authored-by: charbelrami <charbel.f.rami@gmail.com>
2022-08-08 22:39:40 +00:00
noredinkbot-duet[bot]
be86b03eb9
Merge #1019
1019: A11-1242 remove `aria-describedby` when `disclosure` is not rendered in the RadioButton r=charbelrami a=charbelrami

`aria-describedby` references [disclosure](https://package.elm-lang.org/packages/NoRedInk/noredink-ui/latest/Nri-Ui-RadioButton-V4#disclosure) and should be added when disclosure is rendered.

Co-authored-by: charbelrami <charbel.f.rami@gmail.com>
2022-08-08 22:34:25 +00:00
Erik Feeley
7f6c4835e5 Adds disabled and loading attributes/config
This implements similar attributes to how it is being done over in
TextInput.
2022-08-08 16:31:46 -04:00
Tessa
a611b12cc0
Merge pull request #1016 from NoRedInk/dansby/modal-zindex
Increase z-index of modals
2022-08-08 13:28:33 -07:00
charbelrami
bb2c5b53de replace classList [] with Extra.none 2022-08-08 16:57:19 -03:00
charbelrami
a8915aa051 add type="button" to tab only when it's a button 2022-08-08 11:05:42 -03:00
charbelrami
fd7c9746dc add describedby only if disclosure is rendered 2022-08-05 18:03:02 -03:00
charbelrami
a0dd686510 add type="button" to tabs 2022-08-05 14:24:30 -03:00
noredinkbot-duet[bot]
c890b60fd0
Merge #1014
1014: A11-1247 add `role="switch"` to Switch checkbox r=charbelrami a=charbelrami

Makes the SR interpret the Switch checkbox input as a switch (on/off as opposed to checked/unchecked)

<img width="704" alt="Screen Shot 2022-08-04 at 17 37 54" src="https://user-images.githubusercontent.com/5647344/182948116-d8b739e1-aa95-4582-a63d-dd0d2364f928.png">


Co-authored-by: charbelrami <charbel.f.rami@gmail.com>
Co-authored-by: Tessa Kelly <tessakkelly@gmail.com>
2022-08-05 16:37:42 +00:00
Tessa Kelly
a4325b3b8f Bump accessible html version 2022-08-05 09:23:18 -07:00
charbelrami
a214594bce use Accessibility.Styled.Role's switch helper 2022-08-05 13:13:08 -03:00
Ben Dansby
9e405c9ed5 up zindex 2022-08-04 16:44:25 -07:00
noredinkbot-duet[bot]
1f9cfcac06
Merge #1015
1015: A11-1241 remove `aria-hidden` from Menu r=tesk9 a=charbelrami

removes `aria-hidden` from the menu because we already have `display: none`

Co-authored-by: charbelrami <charbel.f.rami@gmail.com>
2022-08-04 21:44:55 +00:00
charbelrami
4e38a32761 remove aria-hidden from menu 2022-08-04 18:09:47 -03:00
charbelrami
2a0a9ceaf9 add role="switch" to Switch checkbox 2022-08-04 17:33:07 -03:00
noredinkbot-duet[bot]
0c0eec7e83
Merge #1013
1013: ask Bors to check the tests r=caiquefigueiredo a=BrianHicks

This sets up Bors, which will automatically merge PRs that pass all the checks. It works like Homu did, but batches PRs into bigger groups for efficiency's sake.

Once this merges, you can talk to it the same way we do `@NoRedInkBot` in the monorepo, except we call it "duet" instead (for now... we just don't want to cause a conflict with two merge robots running on the same triggers!) So for example, `duet r+` will add the PR to the merge batch, `duet r-` will remove it, et cetera.

Fixes FXN-1289


Co-authored-by: Brian Hicks <brian@brianthicks.com>
2022-08-02 20:46:23 +00:00
Brian Hicks
4fe599e3fe
ask Bors to check the tests 2022-08-02 15:15:40 -05:00
Ben Dansby
e093a90520
Merge pull request #1012 from NoRedInk/dansby/center-text
Center Page text
2022-08-02 09:39:47 -07:00
Tessa Kelly
097e360917 re-format 2022-08-02 09:36:12 -07:00
Ben Dansby
48b776051c center text 2022-08-02 07:05:01 -07:00
Ben Dansby
4f4b20784d
Merge pull request #1009 from NoRedInk/dansby/checkbox-bg
Remove white background from checkbox
2022-07-29 10:28:25 -07:00
Ben Dansby
0e6e6b33e9 remove bg 2022-07-29 07:06:39 -07:00
Tessa
4ce609dae3
Merge pull request #1008 from NoRedInk/bump-checkbox
Bump for Checkbox.V6, Checkbox css attributes, noredinkMonochrome
2022-07-28 15:29:42 -07:00
Tessa Kelly
961de8cd1c Bump 2022-07-28 15:23:43 -07:00
Tessa
645c6512e3
Merge pull request #1005 from NoRedInk/tessa/modal
Fix Modal focus
2022-07-28 08:50:39 -07:00
Ben Dansby
8b3caa6bb0
Merge pull request #1007 from NoRedInk/dansby/sort-arrow
New sort arrows
2022-07-28 08:07:34 -07:00
Micah Hahn
2c61b463f6
Merge pull request #1006 from NoRedInk/fix-header
we're on SortableTable version 3
2022-07-28 10:28:25 -04:00
Ben Dansby
3dadb69a9a new arrows 2022-07-28 07:24:25 -07:00
Brian Hicks
ff24337807
we're on SortableTable version 3 2022-07-28 08:39:09 -05:00
Tessa Kelly
04c3b3e21f Fix test 2022-07-27 17:31:22 -07:00
Tessa Kelly
20639b4ecd Elm review 2022-07-27 16:42:45 -07:00
Tessa Kelly
f2decf2835 Fix focus 2022-07-27 16:39:21 -07:00