-
Notifications
You must be signed in to change notification settings - Fork 83
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: sethvargo The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/assign @whaught |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I love it!
/lgtm
The one odd thing here is that the menu goes to settings#general It causes the page to load the first tab and quickly switch to the second. The user sees a weird flicker every time they load the page. Maybe just make 'General' the first tab? |
I think we can set "active" on that first tab? We have to wait for the dom to load to pick the right tab |
This updates the UX on the realm settings page to be tabular. There's currently a lot of settings and they aren't logically grouped - this changes that to group various settings within a tab.
Release Note