-
Notifications
You must be signed in to change notification settings - Fork 16
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
ui to explain follow #164
ui to explain follow #164
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, I couldn't test it in OPIN because it has its own FollowButton. I left two small questions, feel free to ignore.
<span className="dropdown-menu" aria-labelledby="follow-dropdown"> | ||
<button className="dropdown-item select-item" onClick={this.disableFollow.bind(this)}> | ||
{!this.state.followed ? <i className="fa fa-check select-item-indicator" aria-hidden="true" /> : null} | ||
{django.gettext('Unfollow')} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't this be named Not following
and the other option Following
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Valid point, thanks!
<i className="fa fa-caret-down" aria-hidden="true" /> | ||
</button> | ||
<span className="dropdown-menu" aria-labelledby="follow-dropdown"> | ||
<button className="dropdown-item select-item" onClick={this.disableFollow.bind(this)}> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is the class name select-item
not a bit over generic?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I had something like follow
before which I found too specific. You have any suggestions for something in between?
c4f9226
to
bf8561d
Compare
Clicking the button now opens a dropdown where the specific action is explained.
This is breaking because it requires additional styling. See liqd/a4-meinberlin#728. At least setting
is recommended, since I couldn't use any block elements due to then invalid html.