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
[datasource editor] Only one click target for edit action #8495
[datasource editor] Only one click target for edit action #8495
Conversation
title={datasource.name} | ||
className="label label-default label-btn m-r-5" | ||
bsSize="sm" | ||
id="datasource_menu" |
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.
would be better to use className
instead of id
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.
this id
was from old code. i think it's required by react bootstrap:
https://react-bootstrap.github.io/components/dropdowns/#dropdown-button-props
padding-left: 8px; | ||
padding-right: 8px; | ||
} | ||
#datasource_menu .caret { |
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.
add blank line
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.
fixed.
97a644f
to
8ed17ce
Compare
Codecov Report
@@ Coverage Diff @@
## master #8495 +/- ##
==========================================
- Coverage 66.61% 66.61% -0.01%
==========================================
Files 449 449
Lines 22610 22609 -1
Branches 2367 2366 -1
==========================================
- Hits 15061 15060 -1
Misses 7411 7411
Partials 138 138
Continue to review full report at Codecov.
|
(cherry picked from commit acad66d)
CATEGORY
Choose one
SUMMARY
Some of our users didn't know how to find all options for edit datasource, since they didn't realize the datasource name and caret are different label buttons.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
Before:
After:
TEST PLAN
ADDITIONAL INFORMATION
REVIEWERS
@michellethomas @etr2460 @john-bodley