Skip to content
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

Merged
merged 29 commits into from
Jul 3, 2024
Merged

FAQ Page #17

merged 29 commits into from
Jul 3, 2024

Conversation

jonha1
Copy link
Contributor

@jonha1 jonha1 commented Apr 11, 2024

draft of the new FAQ page

closes issue#16

jonha1 added 9 commits April 9, 2024 16:43
changing image icons and updating packages
- starting to develop QNA page
- write descriptions
- needs styling
- need to fix the icon for speaking rate
- needs to look better and center title more
- work on icons
buttons now work and FAQ page still nedds more content
- still need to update FAQ page
@jonha1 jonha1 marked this pull request as draft April 11, 2024 04:07
@jonha1 jonha1 marked this pull request as ready for review April 11, 2024 04:08
@steinbro
Copy link
Member

steinbro commented Apr 14, 2024

Thanks for your work on this! A few items:

  1. The question mark button falls off the side of the screen, at least when I use the mobile display simulator in desktop Firefox:

I'm sure there's better use of screen real estate, like a second row of buttons, but perhaps that should be a separate pull request. For now, just make sure the button is actually visible without horizontal scrolling.

  1. The home button isn't rendering with the expected width:
  1. The diff for the PR shows reversions of some upstream changes. I'm sure this was unintentional, but make sure your branch accurately reflects only the relevant changes. Since I think this same branch was already merged earlier, it might be cleanest to fork a new branch off of main and apply your changes onto that.

@jonha1
Copy link
Contributor Author

jonha1 commented Apr 22, 2024

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.

@steinbro

@steinbro
Copy link
Member

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.

@jonha1
Copy link
Contributor Author

jonha1 commented Apr 22, 2024

I'll work on those as soon as possible.

If future developers would like to know how soundscape would look on Safari on other IOS devices, should I include documentation? If so, where would you like me to include documentation on this?

Below are screenshots on how soundscape looks on an iPhone 8. Im not sure if we can see if there are more updated options for choosing an iPhone but I will look into it and provide feedback.
Screen Shot 2024-04-21 at 11 35 05 PM

Screen Shot 2024-04-21 at 11 36 36 PM

In the first image, the Google voices don't appear in the voice dropdown.

@steinbro
Copy link
Member

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.

@jonha1
Copy link
Contributor Author

jonha1 commented May 2, 2024

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.

@steinbro
Copy link
Member

steinbro commented May 4, 2024

Looks and sounds good!

@jonha1
Copy link
Contributor Author

jonha1 commented Jul 2, 2024

I tested my branch and it should be ready for a merge.

@steinbro
Copy link
Member

steinbro commented Jul 3, 2024

It needed a good bit more cleanup, but it's finally ready for merge :)

@steinbro steinbro merged commit 07de9ff into soundscape-community:main Jul 3, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants