@lbrabec, I have done some minor changes, please have a look.
The more I look at the timeline description, the more it is distracting. Could you please try to hide it under question mark icon with tooltip on hover? I think this would look better.
Blockerbugs stats shows "Fedora 0 blockers and FEs" instead of "Fedora 35 blockers and FEs", please fix this issue.
With regard to design, I have some ideas:
table-bordered table-striped
@jskladan Do you have any comments or ideas wrt to landingpage design?
Apart of what @lbrabec said (once that's done, I'll most probably have more tweaks to be done, but I'd like to see where that takes us, I'd like to have the "settings" button moved to the header, and the Have some spare time? Click on below button to contibute together with the Explore button replaced with just a Click here, and help us make Fedora better! button.
Have some spare time? Click on below button to contibute
Explore
Click here, and help us make Fedora better!
thx!
@jskladan, Regarding the second point, we had a UX feedback report and it was mentioned to avoid long texts in the button. That may lead to a bad UX. Please let me know if I should proceed with this change?
1 new commit added
moved settings button to navbar
@jskladan, @lbrabec I have implemented some changes, please have a look.
The more I look at the timeline description, the more it is distracting. Could you please try to hide it under question mark icon with tooltip on hover? I think this would look better. Done, it look definitely better. Blockerbugs stats shows "Fedora 0 blockers and FEs" instead of "Fedora 35 blockers and FEs", please fix this issue.
The more I look at the timeline description, the more it is distracting. Could you please try to hide it under question mark icon with tooltip on hover? I think this would look better. Done, it look definitely better.
Done.
in all info cards set a bottom margin to titles so the gap between the title and content is visually the same as gap between top border and title
Not implemented yet.
make the vertical spacing between info cards larger so it matches horizontal spacing
in "Fedora QA Meeting Minutes" info card, add padding so the content is not so close to the border (try to match the padding of "Meetings and testdays" info card)
Gap is larger, but still not the same as the gap between border and "calendar" in "Meetings and testdays" info card. Please try to make the gap the same.
parse the date in "Latest minutes from 2021-07-12-15.01" and show it in human friendly way
in "Fedora blockers and FEs" info cards, there is indent in the table headers, remove it so the text is the same distance from left border as contents of other rows
as @jskladan mentioned in #73, blocker info cards looks too "table-y", maybe try to remove table-bordered table-striped from the tables
Look definitely better, WDYT @jskladan ?
I'd like to have the "settings" button moved to the header,
and the Have some spare time? Click on below button to contibute together with the Explore button replaced with just a Click here, and help us make Fedora better! button.
Done, but I think the left and right padding is too large.
Please, implement the remaining.
@jskladan do you have any more comments?
added some padding to the boxes
@lbrabec I have implemented the changes you have asked for, please have a look.
There is padding-left on calendar event title which makes it indented more than the time, please fix this.
Otherwise, I like it!
I didn't notice that my bad, I'll change it.
added padding to minutes card
I merged #83 and this PR now has conflicts, please resolve them.
Sure thing I'll resolve them
rebased onto 7009dcc006edec1f9df8e3549f441894a89bff3a
@lbrabec solved conflicts please have a look!
Thanks! Merging.
Pull-Request has been merged by lbrabec
@lbrabec, I have done some minor changes, please have a look.