The roadmap page is not uniform when displaying multiple milestones. This makes it harder to read as whole. There should also be a key describing each column for each milestone, and the spacing should be uniform across all milestones (like a table)
I am working on this, and should have a new pull-request by tomorrow...
@mreynolds I did some snooping :)
Even if you make all columns except the first one as nowrap, the columns wouldn't align between different milestones. To fix this, apply a width: 100% on all the tables, and all the first columns.
nowrap
width: 100%
Something like this:
table { width: 100%; } td:first-child { width: 100%; }
And the result is a nicer looking Milestone Roadmap page:
Partially stolen from http://stackoverflow.com/questions/3665512/html-table-column-align-right :)
@mreynolds I did some snooping :) Even if you make all columns except the first one as nowrap, the columns wouldn't align between different milestones. To fix this, apply a width: 100% on all the tables, and all the first columns. Something like this: table { width: 100%; } td:first-child { width: 100%; } And the result is a nicer looking Milestone Roadmap page: Partially stolen from http://stackoverflow.com/questions/3665512/html-table-column-align-right :)
@mreynolds I did some snooping :) Even if you make all columns except the first one as nowrap, the columns wouldn't align between different milestones. To fix this, apply a width: 100% on all the tables, and all the first columns. Something like this: table { width: 100%; }
td:first-child { width: 100%; }
Nice! I'll try this out and add a new commit. Thanks!
@cep - adding "width: 100%" to each table and
@mreynolds I just tested it in the browser by applying the styles using jQuery.
It may not work if your styles are overrided by the defaults, which I suspect might be the case. Try specifying them as !important in the CSS and see if that works?
!important
And do note that only the first td in each row should get the width: 100%.
td
Sorry I don't know jquery/html/css very well, but I do not see any "pagure" class being used for the table or rows. All i see is this:
<table id="pagure-issues-list-{{ index }}" class="table table-hover table-striped m-b-0">
Here's a patch that fixes the table layouts across milestones. Based off your latest commit in PR#1724.
I'm sorry, but it still is not working for me (I see no difference after applying your patch): See the 1.2.11 milestone from my screenshot:
You might need to restart the development server and force-reload the pages (Shift + F5 or Ctrl + F5) in your browser to see the changes. The static files might be cached.
Shift
F5
Ctrl
No luck. I did restart my dev server, and I did do a shift-reload(and ctrl f5) on the page.
@pingou: here is a screen shot showing the status/tag/milestone wrapping
Anyway I prefer keeping each criteria on a separate line
I think status should have some icon, even if its generic
Alright https://pagure.io/pagure/pull-request/1724 has been merged, fixing this ticket :)
@pingou changed the status to Closed
Closed
Metadata Update from @lslebodn: - Issue tagged with: IDM