-
Notifications
You must be signed in to change notification settings - Fork 12
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
feat: new account page, new nav section, smaller homepage #67
Conversation
This restores commit f122ddd
@silviamazzoni, Please take a look at this new index page and top section. It's changed since last time. At least during today, you can preview it on https://designsafeci-dev.tacc.utexas.edu/user-guide/. |
Wes, I am happy to meet. The main note is that the section titles, such as DesignSafe Essentials, Data Depot, Tools and Apps, etc should be aligned left so that we know they are not a link. a different format could work. |
also, clicking on a section in the Use Cases overview should open up the corresponding item in the nav... |
Also, there is still too much empty space at the top, both in the nav and in the page. |
also, when I clikc on an item in the nav, it scrolls the left nav up so that that item is that at the top... very disturbing and makes it hard to visualize the full section. |
Understood.
Known issue; in progress: Other feedback seems external to the problem being solved (which is adding "Getting Started" and restoring lost content to get help for account). I gather it for reference, and give answers.
This is a reasonable opinion. Although, in one sense, they are aligned left because the text is aligned left; the arrow is outside the alignment. Independent of this content-focused PR, feel free to continue to remind me about this element. (I forget what designer's feedback was when last I mentioned it.)
Uncertain. I am unfamiliar with pre-existing content.
Uncertain. I presume "Overview" pages are new additions and have not been viewed as a standard per section. Update:
Design concern. Seems irrelevant to the problem this PR solves. Feel free to open a new issue.
Aye, it does. Sometimes it can not scroll item to top, and instead scrolls nav up as far as it can, with relevant item appearing in the middle of the nav. I am continuously aware of this issue, and hope it is fixed on its own after https://github.com/TACC/TACC-Docs/issues/53. |
I updated my original response with answers to feedback I missed.
Design concern. Seems irrelevant to the problem this PR solves. Feel free to open a new issue.
Aye, it does. Sometimes it can not scroll item to top, and instead scrolls nav up as far as it can, with relevant item appearing in the middle of the nav. I am continuously aware of this issue, and hope it is fixed on its own after https://github.com/TACC/TACC-Docs/issues/53. |
Merging based on informal approval. I made these changes under @heddabedda's direction. I got review from @silviamazzoni (colleague), @heddabedda's (new author), and one project stakeholder. Original content is basically as is, just moved around. I will request a second stakeholder to scan it after deploy. |
Overview
Update homepage. Move account access to new page. New top nav section.
Status
Important
Pending stakeholder review (internal, private).
Related
Changes
Testing
UI
After enhance: rework as Getting Started & Account Help
Getting Started
Account Help
Before enhance: rework as Getting Started & Account Help
Homepage & Nav
Account Access