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

Change collapsed arrow direction for lists and editor widgets #1059

Merged
merged 2 commits into from
Jan 29, 2018

Conversation

Doocey
Copy link
Contributor

@Doocey Doocey commented Jan 29, 2018

- Summary

Change the default direction of the chevron icon on in-editor widgets. Widgets such as the Image and YouTube embed features now have a right-facing arrow icon when in collapsed state signalling encased content, rather than an up (^) icon.
This change address enhancement suggestion #1057 to improve the UX of the editor widgets.

- Before:
default-behaviour

- New Behaviour:
preferred-behaviour

- Description for the changelog

In-editor widget icon direction change for better UX when in a collapsed state.

- On submission of pull request

cat

@verythorough
Copy link
Contributor

verythorough commented Jan 29, 2018

Deploy preview for netlify-cms-www ready!

Built with commit dff9939

https://deploy-preview-1059--netlify-cms-www.netlify.com

@verythorough
Copy link
Contributor

verythorough commented Jan 29, 2018

Deploy preview for cms-demo ready!

Built with commit dff9939

https://deploy-preview-1059--cms-demo.netlify.com

Copy link
Contributor

@erquhart erquhart left a comment

Choose a reason for hiding this comment

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

Totally agree, thanks!

Copy link
Contributor

@tech4him1 tech4him1 left a comment

Choose a reason for hiding this comment

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

Perfect, thanks!

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.

4 participants