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

Very rough reorganization of the storage UI #1730

Closed

Conversation

ancorgs
Copy link
Contributor

@ancorgs ancorgs commented Nov 5, 2024

Completely temporal, just in case it's useful for someone.

cabrear-david-feo

There must be better options than a raw ExpandableSection.

cabrear-david-feo-ext

@dgdavid
Copy link
Contributor

dgdavid commented Nov 7, 2024

We have been trying to shape this new page for a couple of days now. Having other things to do in the middle, of course. Despite all the things I have said "offline", I still having doubts on how to proceed. I have slightly written the rationale at https://trello.com/c/nOUYvYIY/3873-8-first-prototype-of-the-new-ui#comment-672c738f86a51a5abbd1c842 (private link). Summarizing it a bit for a broader audience: it's so challenging to come up with a proposal that put a11y on the front mainly because of time. Somehow I feel we're in a hurry because a lot of things has changed in the backend that now needs a front-end for the users interact with.

That said, a proposal I had in mind still not convincing me, although I like it more than the @ancorgs versions above. It basically needs a lot of more tests and work to ensure it worth it in terms of a11y, reason while I fear we will go an step backward and work in the mode we were working: a11y later. I guess it is a matter of gathering more and more knowledge, working on it.

Screen Shot 2024-11-07 at 07 49 24
Screen Shot 2024-11-07 at 08 09 28

Screen Shot 2024-11-07 at 08 09 25
Screen Shot 2024-11-07 at 08 12 16

If the menu shown in the screenshots above were accessible, even the "Change device" button could be moved to the "More actions", renaming the latest to something more general.

But most probably we will go for the @ancorgs proposal for the initial version because I myself have doubts about the accessibility of the above research.

@ancorgs
Copy link
Contributor Author

ancorgs commented Nov 8, 2024

Closed in favor of #1742

@joseivanlopez joseivanlopez force-pushed the storage-show-config branch 2 times, most recently from 00e3b01 to 26f5fe7 Compare November 11, 2024 15:27
@joseivanlopez joseivanlopez deleted the branch agama-project:storage-show-config November 11, 2024 15:58
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.

3 participants