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

Adjust open indicator margin to align #2306

Closed
wants to merge 2 commits into from
Closed

Conversation

drewvolz
Copy link
Collaborator

Before After
closed-before closed after
open-before open-after

@hawkrives
Copy link
Owner

hawkrives commented Oct 10, 2018

It now looks unaligned when the edit mode is closed, IMO

@drewvolz
Copy link
Collaborator Author

@hawkrives I have fixed the issue you've pointed out in the closed state.

@hawkrives
Copy link
Owner

hawkrives commented Oct 10, 2018

It still doesn't look aligned.

hawkrives added a commit that referenced this pull request Oct 10, 2018
With the new area picker interface, which has giant [X] buttons attached
to each entry, I don't think that we need the (x) buttons on the
individual areas any longer.

Plus, they're buggy wrt. layout currently, ever since we added extra
padding to FlatButton&co.

Closes #2306
@hawkrives hawkrives deleted the open-indicator-margin branch October 11, 2018 13: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.

2 participants