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

Custom buttons not visible in second row #3365

Merged

Conversation

lgalis
Copy link
Contributor

@lgalis lgalis commented Jul 6, 2015

@dclarizio - please review

When having multiple VM custom button groups that would cause them to be displayed on the next line, they are not visible until the window is resized.
This change modifies the height for the cell in the layout to allow for the initial display.

https://bugzilla.redhat.com/show_bug.cgi?id=1234987

1234987_after
1234987_before

@chessbyte chessbyte changed the title Custom butons not visible in second row Custom buttons not visible in second row Jul 7, 2015
@chessbyte
Copy link
Member

@lgalis Please rebase and re-push.

When having multiple VM custom button groups that would casue them to be displayed on the next line, they are not visible until the window is resized.
 This change modifies the height for the cell in the layout to allow for the initial display.

 https://bugzilla.redhat.com/show_bug.cgi?id=1234987
When having multiple VM custom button groups that would cause them to be displayed on the next line, they are not visible until the window is resized.
 Modify the function that adjusts the height of the taskbar as needed so that the custom buttons in the second row are visible .

 https://bugzilla.redhat.com/show_bug.cgi?id=1234987
@lgalis lgalis force-pushed the custom_butons_not_visible_in_second_row branch from 20121a9 to c50dd69 Compare July 8, 2015 14:07
@miq-bot
Copy link
Member

miq-bot commented Jul 8, 2015

Checked commits lgalis@b568c8e .. lgalis@c50dd69 with rubocop 0.32.0
1 file checked, 0 offenses detected
Everything looks good. ⭐

@dclarizio
Copy link

Verified fix.

dclarizio pushed a commit that referenced this pull request Jul 9, 2015
…ond_row

Custom buttons not visible in second row
@dclarizio dclarizio merged commit 89f6310 into ManageIQ:master Jul 9, 2015
@dclarizio dclarizio added this to the Sprint 26 Ending July 13, 2015 milestone Jul 9, 2015
@romanblanco
Copy link
Member

@lgalis after merging this PR i'm getting error:

 Uncaught ReferenceError: miqResizeTaskbarCell is not defined
 miqInitToolbars @ miq_dhtmlxtoolbar.js:11miqOnLoad @ miq_application.js:47onload @ show_list:102

2015-07-10-171631_1920x1080_scrot
Steps to recreate:
Infrastructure → Resource pools

@lgalis
Copy link
Contributor Author

lgalis commented Jul 13, 2015

Fixed in #3422

dclarizio pushed a commit that referenced this pull request Jul 13, 2015
tenderlove added a commit to tenderlove/manageiq that referenced this pull request Jul 13, 2015
* master: (37 commits)
  fix border color around a required select box.
  Fixed typo in .codeclimate.yml
  Added classes to list of CLASSES_THAT_PARTICIPATE_IN_RBAC
  providerForemanFormController - add $scope.model pointing to the real model
  checkchange.js - use lodash for deep copy and deep compare, check that modelCopy exists
  Add .codeclimate.yml
  Custom button fix for ManageIQ#3365 causes js exception in non-explorer tabs
  add lodash.js
  kubernetes: persist env section in container definition
  Changed tooltip text when Adding new VM Policy
  Make PidFile#pid return nil for non-numeric pid file contents
  Unable to add Infrastructure and cloud providers
  Set available locales on all threads.
  Fixed code to use local variable instead of @record.
  Increase connection pool to 5.
  Switch webserver from thin to puma.
  Angular version upgrade to 1.3.15
  Give the blocked connection some time to notice
  Remove threadsafe Stack class for sqlserver support back in 2008.
  convert style1 class tables to form-horizontal class layout Issue: ManageIQ#3139
  ...
tenderlove added a commit to tenderlove/manageiq that referenced this pull request Jul 13, 2015
* master:
  fix border color around a required select box.
  Fixed typo in .codeclimate.yml
  Added classes to list of CLASSES_THAT_PARTICIPATE_IN_RBAC
  providerForemanFormController - add $scope.model pointing to the real model
  checkchange.js - use lodash for deep copy and deep compare, check that modelCopy exists
  Custom button fix for ManageIQ#3365 causes js exception in non-explorer tabs
  add lodash.js
@lgalis lgalis deleted the custom_butons_not_visible_in_second_row branch August 3, 2015 18:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants