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

UI updates for How-To page #556

Open
subaha-cob opened this issue Jul 7, 2021 · 15 comments
Open

UI updates for How-To page #556

subaha-cob opened this issue Jul 7, 2021 · 15 comments

Comments

@subaha-cob
Copy link
Contributor

subaha-cob commented Jul 7, 2021

Currently, How-To pages use a tabs, located below the header, to differentiate between sections. UX research has demonstrated that users often miss these tabs as they scroll down the page. To address this issue, the tabs should be replaced by series of drawers, as shown in the wireframe below:

Screen Shot 2021-01-14 at 9 18 22 AM

The current UI is shown below for reference:

Screen Shot 2021-01-14 at 9 19 02 AM

Complete specs can be found here: https://www.boston.gov/sites/default/files/embed/a/annotated_wireframes.pdf

The "carrot" indicating the state (open/closed) of the drawers has also been shown to be confusing for users, and should be replaced by a +/- symbol indicating open/closed state, respectively. The content of the new drawers is identical to the content previously housed in each tab.

This looks different from the wireframes now. With the $charles-blue and $optimistic-blue-dark colors instead for hover and open states.

@subaha-cob subaha-cob removed this from Proposed in Fleet and Zero Height updates Jul 9, 2021
@subaha-cob subaha-cob added this to Needs Triage in Legacy CSS Removal - Fellows via automation Jul 9, 2021
@subaha-cob subaha-cob moved this from Needs Triage to Priority in Legacy CSS Removal - Fellows Jul 9, 2021
@carolyn-wang carolyn-wang moved this from Priority to In Progress in Legacy CSS Removal - Fellows Jul 19, 2021
@subaha-cob
Copy link
Contributor Author

There are 3 states: Open, close and hover states:

Screen Shot 2021-07-19 at 2 41 32 PM

@subaha-cob
Copy link
Contributor Author

@carolyn-wang you have had this task for a while now. Where are we at with this task?

@phillipbentonkelly
Copy link
Contributor

I pulled down the branch and from what I see it still needs work, not all 3 states are there. We can discuss/assists later, but preferably separate from the 1pm meetings

Screen Shot 2021-08-06 at 11 38 56 AM

@carolyn-wang
Copy link
Contributor

Branch #582 is outdated for this task, do not use. I created a new branch (#583) so that the how-to tabs would be separate from the group of links

@rachelsater
Copy link

@subaha-cob hiya! Can you please let me know what link to test with?

@subaha-cob
Copy link
Contributor Author

@rachelsater rachelsater moved this from Ready for Review to Done in CSS and Template Updates Oct 26, 2021
@rachelsater
Copy link

@duffy-james this looks good to me, what do you think?

@duffy-james
Copy link

@rachelsater @subaha-cob the sidebar and the drawers look a little too close together compared to the original:

image

Current version:

image

The text in the header here in the first step looks slightly gray instead of black:

image

Current version:

image

One last thing, on mobile, the title of each step is still appearing as it would normally on desktop. Also, I think the text underneath each step is slightly larger:

image

Current version:

image

There's a ton of extra white space at the bottom of the last component on the page and the sidebar section, but that was also in the original:

image

@rachelsater rachelsater moved this from Done to In Dev in CSS and Template Updates Nov 9, 2021
@rachelsater rachelsater moved this from In Dev to Ready for Review in CSS and Template Updates Nov 16, 2021
@rachelsater
Copy link

@subaha-cob Looks like this is still an issue. Mobile has a lot of extra spacing and the Back To Top box is covering text
image

Please Keep In Mind is two different colors

image

image

@rachelsater rachelsater moved this from Ready for Review to In Dev in CSS and Template Updates Nov 16, 2021
@subaha-cob
Copy link
Contributor Author

@rachelsater this is not a bug we introduced, but it is a bug that is currently on production. This will be fixed in this project.

@rachelsater
Copy link

@subaha-cob both the mobile and "keep in mind" issues?

@subaha-cob
Copy link
Contributor Author

@rachelsater That is also what it looks like in production. We can also fix in this project

@rachelsater
Copy link

@duffy-james what do you think?

@subaha-cob subaha-cob moved this from In Dev to Done in CSS and Template Updates Jun 2, 2022
@subaha-cob subaha-cob moved this from Ready for Review to Done in Fleet and Component Updates Jun 2, 2022
@subaha-cob subaha-cob moved this from Ready for Review to Done in Legacy CSS Removal - Fellows Jun 2, 2022
@duffy-james
Copy link

@subaha-cob the title text is still appearing really large on mobile:

image

Ideally the text size would be much smaller on mobile (an h3 instead of an h2), and the numbered step would be next to the title of each step, not above it.

@duffy-james duffy-james reopened this Jun 2, 2022
Legacy CSS Removal - Fellows automation moved this from Done to Priority Jun 2, 2022
Fleet and Component Updates automation moved this from Done to To Do Jun 2, 2022
CSS and Template Updates automation moved this from Done to To Do Jun 2, 2022
@subaha-cob subaha-cob moved this from To Do to Done in CSS and Template Updates Aug 22, 2022
@subaha-cob subaha-cob moved this from To Do to Done in Fleet and Component Updates Aug 22, 2022
@subaha-cob subaha-cob moved this from Priority to Done in Legacy CSS Removal - Fellows Aug 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment