-
Notifications
You must be signed in to change notification settings - Fork 2
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
FAQ Page #17
FAQ Page #17
Conversation
changing image icons and updating packages
- starting to develop QNA page
- write descriptions - needs styling - need to fix the icon for speaking rate
buttons now work and FAQ page still nedds more content
- still need to update FAQ page
…cape-web-client into header-changes
Thank you for the feedback. I finished the changes you requested in 1 and 2 earlier. I also plan to investigate how the page looks on safari. For #3 about the upstream changes, I wanna clarify if it would be easier if you are suggesting if I should make a new branch from main and put all the changes I have done onto the new branch I just created. |
Great, thanks! I can confirm the ? button now fits on the screen, but the buttons now have no visible space between them. This isn't great for accessibility, since users who can't make out the button labels may navigate based on the nth button on the screen, which is tough to distinguish now. I'd suggest making the voice drop down narrower; it's currently just showing the first few characters of the voice name, which isn't super helpful (especially on Windows, where all the voice names star with "Microsoft"). As for the upstream changes, yes, I was suggesting one possible fix that would save you some git merging headaches is to just start a new branch . This needs to be fixed because, for example, the redesigned speed controller no longer works because the changes in app/js/visual/voicecontrols.js have been inadvertently reverted in this branch. |
Eh, no need to document how the app renders in the Safari mobile rendering mode. Ideally, these images will quickly become out-of-date anyway. |
I just fixed the spacing in the header, let me know if it's good. If no other changes are needed, then I will make a new branch and merge those changes. |
Looks and sounds good! |
- need to refactor link to help page
This reverts commit 39f47c3.
I tested my branch and it should be ready for a merge. |
It needed a good bit more cleanup, but it's finally ready for merge :) |
draft of the new FAQ page
closes issue#16