horizon.less does not use bootstrap mixins

Bug #1287616 reported by Maxime Vidori
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Dashboard (Horizon)
Invalid
Undecided
Maxime Vidori

Bug Description

Horizon use Bootstrap to provide an easy way to design the application. Bootstrap has some utils functions called mixins, these functions avoid the use of prefixed css properties. In the file horizon.less those mixins are not used and can lead to issues on dedicated browsers, in addition it reduces the number of line to maintain.

Furthermore, a lot of features which could be handled by css are made in javascript with jquery, it could be interesting to migrate this logic to css and reduce the footprint of the javascript in horizon.

Changed in horizon:
assignee: nobody → Maxime Vidori (maxime-vidori)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on horizon (master)

Change abandoned by David Lyle (<email address hidden>) on branch: master
Review: https://review.openstack.org/76284
Reason: This review is > 4 weeks without comment, and failed Jenkins the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a 'recheck' comment to get fresh test results.

Revision history for this message
David Lyle (david-lyle) wrote :

horizon.less was converted to horizon.scss

Changed in horizon:
status: In Progress → Invalid
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.