• Fatih Acet's avatar
    Merge branch '23167-min-row-height' into 'master' · ecd7e99a
    Fatih Acet authored
    Resolve "Build/Pipeline/Environment List Inconsistency Row Height"
    
    #### What does this MR do?
    * Changes generic table class from `builds` to `ci-table`. It was getting confusing using `builds` for `builds`, `pipelines`, `environments`
    * Sets height on builds rows for continuity
    
    #### Are there points in the code the reviewer needs to double check?
    Apparently you can't set `min-height` on any `table` elements but you _can_ set `height`. For some reason, even if the content within happens to start wrapping, the row height will grow instead of letting the content overlap. Which is a good thing. Just kind of weird.
    
    #### Why was this MR needed?
    Different row heights on builds depending on content
    
    #### Screenshots (if relevant)
    Before:   
    ![Screen_Shot_2016-10-10_at_4.49.58_PM](/uploads/a4edb584f95c670f9815a8e5b1d725ee/Screen_Shot_2016-10-10_at_4.49.58_PM.png)
    After:   
    ![Screen_Shot_2016-10-10_at_4.32.03_PM](/uploads/488a84b4ed292fbbb0ea7e372c017ae0/Screen_Shot_2016-10-10_at_4.32.03_PM.png)
    Before:   
    ![Screen_Shot_2016-10-10_at_4.44.23_PM](/uploads/3cbcbada89d1aeb1fea35ea9b851e370/Screen_Shot_2016-10-10_at_4.44.23_PM.png)
    After:   
    ![Screen_Shot_2016-10-10_at_4.42.56_PM](/uploads/0cb573670f60c7fdf54fdb027c95639f/Screen_Shot_2016-10-10_at_4.42.56_PM.png)
    
    ## What are the relevant issue numbers?
    Closes #23167
    
    See merge request !6787
    ecd7e99a
merge_requests.scss 6.23 KB