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

Proposal: Block & area style slider control improvement #1670

Closed
olsgreen opened this issue Dec 18, 2014 · 2 comments
Closed

Proposal: Block & area style slider control improvement #1670

olsgreen opened this issue Dec 18, 2014 · 2 comments

Comments

@olsgreen
Copy link
Contributor

Hey all,

While the introduction of sliders to control block and area padding, margins, etc is great, we've noticed an issue with consistently and easily specifying exact values. Sometimes the slider seems to snap to values above or below what we want. Feedback from users is that it would be great to replace the unit read out with a small input control to allowing for direct input of the pixel values, mock below.

I'm happy to implement, should be an easy one, thoughts?

A go ahead or refusal from the core team is what I'm looking for.

boxes

@aembler
Copy link
Member

aembler commented Dec 18, 2014

Yes we would happily accept this change. Adding input fields is something we've wanted to do for awhile just haven't had a chance. Thanks!

Sent from my iPad

On Dec 18, 2014, at 1:55 AM, Oliver Green notifications@github.com wrote:

Hey all,

While the introduction of sliders to control block and area padding, margins, etc is great, we've noticed an issue with consistently and easily specifying exact values. Sometimes the slider seems to snap to values above or below what we want. Feedback from users is that it would be great to replace the unit read out with a small input control to allowing for direct input of the pixel values, mock below.

I'm happy to implement, thoughts?

A go ahead or refusal from the core team is what I'm looking for.


Reply to this email directly or view it on GitHub.

@olsgreen
Copy link
Contributor Author

Thanks @aembler will implement and reference this in the pull. Cheers.

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

No branches or pull requests

2 participants