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 improvements #75

Open
ArthurClemens opened this issue Nov 12, 2023 · 6 comments
Open

UI improvements #75

ArthurClemens opened this issue Nov 12, 2023 · 6 comments
Labels
enhancement New feature or request

Comments

@ArthurClemens
Copy link

I've create a wireframe/visual design to visualize a potential new user interface for Ash Admin.

The base idea is to give more room to the navigation - to replace the current buttons and selects in the top bar with navigation items in de sidebar.

The screens incorporate ideas for more intuitive interactions by using button labels, such as the disconnect actor buttons.

To prioritize data visibility, I've minimized the color palette, with a couple of brand color touches.

Ash Admin UX.pdf

Single screen:

20231112 Ash Admin UX - User set
@ArthurClemens ArthurClemens added the enhancement New feature or request label Nov 12, 2023
@zachdaniel
Copy link
Contributor

Big fan :)

@jimsynz
Copy link
Contributor

jimsynz commented Nov 13, 2023

Agreed.

@ArthurClemens
Copy link
Author

ArthurClemens commented Nov 13, 2023

Good to hear!

Before moving on, I'd like to get a feeling about functionalities that I've missed, and that impact the layout and interactions.

Thinks I know that are not included yet:

  • Multitenancy
  • Pagination of results
  • Unauthorised error message (or error messages in general)

Any more?

@zachdaniel
Copy link
Contributor

  • Multitenancy is currently implemented by supporting a text box in the top right if any resources support multi tenancy. There isn't currently a way to generically ask a data layer to list possible values for the multi tenancy value, and that would be a performance issue anyway. So basically a text field somewhere in the UI if any resources supports multitenancy
  • pagination is implemented on each table view using relatively standard pagination UIs. There should be an example if you run mix dev and go to a read action. If a resource doesn't support pagination, we limit to 1000 records for safety.
  • error messages are currently dumped in a very ugly way onto the top of a page.

@jimsynz
Copy link
Contributor

jimsynz commented Nov 13, 2023

I know that @joshprice had some thoughts he wants to share - we discussed it internally.

One thing is to talk a bit about the audience, because that helps us decide what's relevant and what isn't. ash_admin has always been more of an ash_super_admin or ash_dev_panel - that's fine if we're happy to keep using it that way. I'm wondering if there's ways we can show more useful information in the "Properties" page; like number of records, storage space used, etc?

@zachdaniel
Copy link
Contributor

zachdaniel commented Nov 13, 2023

I think there are definitely ways to do that, but those will need to come with the overhaul of how an admin ui is actually configured. The idea in the future is to make a DSL that defines an admin UI instead of the "in-the-resource" method we're doing now which is quite obviously (EDIT: in retrospect) a bad idea I think.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants