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

Timeline menu #3827

Merged
merged 6 commits into from
Apr 26, 2018
Merged

Timeline menu #3827

merged 6 commits into from
Apr 26, 2018

Conversation

martinpovolny
Copy link
Member

@martinpovolny martinpovolny commented Apr 23, 2018

Put the Right Menu into Remove menu from the Timeline screen.

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

Changes:

  • remove one level of nesting in ToolbarChooser (so that we can actually read the condition)
  • split method button in EmsCommon (the method was already a if/else/case branching with methods calls, but somehow people started adding in-line code again 😢 😭 )
  • remove extra code from timeline_pressed
  • combination of @layout and @last_action then causes the correct toolbar: timeline_center to be rendered.
  • make the redirect in update_ems_button_cancel more robust (will default to show when given action does not exist).

Steps for Testing/QA

  1. Have a provider (tested with vSphere infra)
  2. Open the provider's details
  3. Select Monitoring/Timelines
  4. Inside Timelines, select Configuration/Edit this XYZ provider this is no longer possible with this PR
  5. Inside the edit dialog, press Cancel

@miq-bot miq-bot added the wip label Apr 23, 2018
@martinpovolny martinpovolny changed the title [WIP] Timeline menu Timeline menu Apr 24, 2018
@martinpovolny martinpovolny changed the title Timeline menu [WIP] Timeline menu Apr 24, 2018
@martinpovolny martinpovolny changed the title [WIP] Timeline menu Timeline menu Apr 25, 2018
@martinpovolny martinpovolny removed the wip label Apr 25, 2018
@miq-bot
Copy link
Member

miq-bot commented Apr 25, 2018

Checked commits martinpovolny/manageiq-ui-classic@8781604~...0c3eca4 with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
4 files checked, 1 offense detected

app/helpers/application_helper/toolbar_chooser.rb

@mzazrivec mzazrivec self-assigned this Apr 26, 2018
@mzazrivec mzazrivec added this to the Sprint 85 Ending May 7, 2018 milestone Apr 26, 2018
@mzazrivec mzazrivec merged commit 82a0615 into ManageIQ:master Apr 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants