Page MenuHomeSoftware Heritage

assets/webapp.css: Improve active nav tab stylesheets
ClosedPublic

Authored by anlambert on Jun 15 2020, 2:53 PM.

Details

Summary

Add a top border of 3px colored with the SWH theme red for nicer display

Before:

After:

Diff Detail

Repository
rDWAPPS Web applications
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

ardumont added a subscriber: ardumont.

Nice.

Isn't it a bit too large though (question based solely from the rendering of the associated image)?

(I would have changed only the color and keep the larger of 1px at first. You know better so fine ;)

This revision is now accepted and ready to land.Jun 15 2020, 3:01 PM

Build is green

Patch application report for D3282 (id=11642)

Rebasing onto bfd6665bf7...

Current branch diff-target is up to date.
Changes applied before test
commit 2d0db7f23a41d02a458a941a4134c659926dc735
Author: Antoine Lambert <antoine.lambert@inria.fr>
Date:   Mon Jun 15 13:54:05 2020 +0200

    assets/webapp.css: Improve active nav tab stylesheets
    
    Add a top border of 3px colored with the SWH theme red for nicer display

See https://jenkins.softwareheritage.org/job/DWAPPS/job/tests-on-diff/183/ for more details.

Isn't it a bit too large though (question based solely from the rendering of the associated image)?

3px is the right value to get a nice selection effect (Github UI uses the same value for instance).