-
Notifications
You must be signed in to change notification settings - Fork 357
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
Make the dragdrop placeholder in the main dashboard visible #2324
Make the dragdrop placeholder in the main dashboard visible #2324
Conversation
@skateman Looks good. Just add this change:
|
95a459e
to
f4c1207
Compare
@miq-bot add_label bug |
@miq-bot add_label fine/no |
@himdel can you please take a look? |
This pull request is not mergeable. Please rebase and repush. |
f4c1207
to
1a80f58
Compare
Checked commit skateman@1a80f58 with ruby 2.3.3, rubocop 0.47.1, and haml-lint 0.20.0 |
LGTM, merging 👍 |
Adding |
Make the dragdrop placeholder in the main dashboard visible (cherry picked from commit 1decfd8) https://bugzilla.redhat.com/show_bug.cgi?id=1511524
Gaprindashvili backport details:
|
@skateman The BZ has cfme-5.8.z flag, but this PR is marked as |
@miq-bot add_label fine/yes |
Make the dragdrop placeholder in the main dashboard visible (cherry picked from commit 1decfd8) https://bugzilla.redhat.com/show_bug.cgi?id=1568168
Fine backport details:
|
The
sortable
module injquery-ui
is not sensitive enough when moving a dashboard widget to the bottom of the page. I was trying to add some temporary extra padding when moving a widget, but that was not working. But as we're moving towards angular we'll drop all thejquery
code and so the issue will disappear. Until then I'm adding this placeholder feature to help the drag & drop process to be more understandable for the user.@epwinchell can you please review the styling of the placeholder box? I'm pretty sure I did it wrong 😉
https://bugzilla.redhat.com/show_bug.cgi?id=1485637