-
Notifications
You must be signed in to change notification settings - Fork 241
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
expansion icon is confusing #310
Comments
Hmm... the right arrow is also what browsers display by default for the This is a
This is the contents!
|
Ah, I thought the down arrow was standard but it's entirely possible I've
just been working too much with Material Design. This is what MD, and by
extension most Google properties, does:
https://material.io/archive/guidelines/components/expansion-panels.html.
…On Tue, 19 Mar 2019, 21:08 Natalie Weizenbaum, ***@***.***> wrote:
Hmm... the right arrow is also what browsers display by default for the
<details> tag, so I had expected it to be the standard. For example:
This is a <details> tag! This is the contents!
Do you have a different suggestion for an expansion indicator?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#310 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AIREeG42GBcMwQ3-eydn_OpqyDtFk7Qwks5vYVG_gaJpZM4b77FV>
.
|
I think the down arrow is pretty clear too... @jina, what do you think? |
standard is right becomes down, but we can switch it to the material style. |
@jina The final decision is up to you, of course. I personally think both look pretty clear. |
http://staging.sass-lang.com/documentation/syntax/parsing
My expectation is that right-arrows mean "play"; so the arrow used to show expandable content for compatibility was confusing for me.
It didn't appeal to me when scanning for how to find out more about compatibility because I was looking for a down-arrow or a hover tooltip. Later, when I did see it and think about it I assumed it was a play button of some sort and only clicked to find out what it did.
The text was updated successfully, but these errors were encountered: