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

New admin reorganization #2005

Merged
merged 2 commits into from
May 20, 2016
Merged

Conversation

zarvox
Copy link
Collaborator

@zarvox zarvox commented May 19, 2016

  • Adds a root admin panel page
  • Divides admin panel pages into Configuration (server settings) and Management (day-to-day operational task support).
  • Moves to breadcrumb-based hierarchical navigation for individual admin pages (which works very nicely for e.g. Admin / Users / Invite)
  • Corrects a good deal of inconsistent capitalization in the product. The standard we (attempt to) follow is to capitalize the first letter of a heading, label, or button, but no other words, unless they are proper nouns. For example, "Sandstorm for Work" and "Sandstorm" are always capitalized as written here.
    • I don't think "Trash" and "Main list" qualify as proper nouns.
    • App market is not a proper noun when we write it on our blog, so I assumed it needn't be here either.
    • I didn't bother fixing capitalization in the old admin panel since I intend to deprecate/remove it Soon™.

admin-panel-root

admin-users-invite-breadcrumbs

@paulproteus
Copy link
Collaborator

paulproteus commented May 19, 2016 via email

@paulproteus
Copy link
Collaborator

Also wow, this is great, looking at the screenshots.

@kentonv kentonv merged commit e4a1aa4 into sandstorm-io:master May 20, 2016
@zarvox zarvox deleted the new-admin-breadcrumbs branch May 20, 2016 19:55
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