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

Restyle Report Filter Screen #2040

Merged
merged 1 commit into from
Aug 31, 2017
Merged

Restyle Report Filter Screen #2040

merged 1 commit into from
Aug 31, 2017

Conversation

epwinchell
Copy link
Contributor

@epwinchell epwinchell commented Aug 29, 2017

This PR standardizes all items into the form-horizontal layout and relabels items for clarity.

Relates to: #1939 and #1974

Old

screen shot 2017-08-29 at 9 46 50 am

New

screen shot 2017-08-30 at 10 44 40 am

@epwinchell
Copy link
Contributor Author

@miq-bot add_label fine/no, technical debt, formatting/styling

@epwinchell
Copy link
Contributor Author

@miq-bot assign @dclarizio

@martinpovolny
Copy link
Member

@epwinchell : please, take a look at the Rubocop issues. Some seem 1/2 should be easy to fix.

@miq-bot
Copy link
Member

miq-bot commented Aug 30, 2017

Checked commit https://github.com/epwinchell/manageiq-ui-classic/commit/c23346de431e4292180fa17bc63657fe98002fcf with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
1 file checked, 7 offenses detected

app/views/report/_form_filter.html.haml

  • ⚠️ - Line 22 - Missing curly braces around a hash parameter.
  • ⚠️ - Line 22 - Prefer to_s over string interpolation.
  • ⚠️ - Line 35 - Avoid more than 3 levels of block nesting.
  • ⚠️ - Line 52 - Missing curly braces around a hash parameter.
  • ⚠️ - Line 52 - Prefer to_s over string interpolation.
  • ⚠️ - Line 58 - Avoid more than 3 levels of block nesting.
  • ⚠️ - Line 63 - Avoid more than 3 levels of block nesting.

@epwinchell
Copy link
Contributor Author

@martinpovolny Done. For the rest, I'm open to suggestions.

@martinpovolny martinpovolny merged commit e11ad29 into ManageIQ:master Aug 31, 2017
@martinpovolny martinpovolny added this to the Sprint 68 Ending Sep 4, 2017 milestone Aug 31, 2017
@epwinchell epwinchell deleted the report_filter branch October 25, 2017 18:40
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.

4 participants