forked from ShelterTechSF/askdarcel-web
-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Development #142
Merged
Merged
Development #142
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* form successful without recaptcha (if my emma form recaptcha is turned off) * add recaptcha to signup form (testing) * lint error * lint error again
Category cards
update resolver dns ip
…es date/time rendering (#101) --------- Co-authored-by: Adrien Young <[email protected]>
--------- Co-authored-by: Ross Chapman <[email protected]>
* Add page wrapper so users can skip to main content on any page. Also adds a hidden ref to focus and unfocus so when user visits a new page the focus element resets * get rid of PageWrapper and wrap router instead * wrap router with main * create skip button that is hidden until focused * remove duplicate code
* search / tlc page sr-only h1 * add hidden h1 to homepage * hidden h2s for a11y on search results page * .sr-only class
* Make category dropdown more accessible * lint error * created MenuHooks and DropdownSubmenu to DRY up code * refactor dropdownsubmenu so categorydropdown can use it. also move categorydropdown to navigation dir * DRY up scss files * lint error * add comments back into hooks * change DropdownMenu name and call hooks inside component rather than as props * rm onclick from Link because changing page removes focus anyway * use DropdownMenu instead of CategoryDropdown * separate scss into modules per component, remove redundant css * lint error * rm old comment * rm redundant handleMenuToggle
* Update NavigationFocusReset only on top-level page changes not search query changes * edit so focus is reset on all page changes except filter and query changes * use location.search instead to only check if query string changes
… href, add isVisualOnly prop to Button (#141)
rosschapman
approved these changes
Aug 19, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.