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

Inserter: Move the inserter at the bottom of the editor to the right #777

Merged
merged 1 commit into from
May 15, 2017

Conversation

youknowriad
Copy link
Contributor

closes #775

This PR adds the position to position the inserter to the right (moving the inserter and the arrow), we could easily add the possibility to left align it as well.

@youknowriad youknowriad added the [Feature] Inserter The main way to insert blocks using the + button in the editing interface label May 12, 2017
@youknowriad youknowriad self-assigned this May 12, 2017
@youknowriad youknowriad requested a review from jasmussen May 12, 2017 11:32
Copy link
Member

@aduth aduth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can't speak to design implications, but the code looks good 👍

@youknowriad
Copy link
Contributor Author

@jasmussen Any thoughts on the design here?

@jasmussen
Copy link
Contributor

Looks good! Perhaps when it's on the right, push it in another few pixels so the left edge of the inserter aligns with the left border of a highlighted block. It's a little tight on the edge at mobile responsive:

screen shot 2017-05-15 at 10 30 34

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Inserter The main way to insert blocks using the + button in the editing interface
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Sidebar menu items overlaps the inserter
3 participants