Sarai-WMDE added a subscriber: Arian_Bozorg.
Sarai-WMDE added a comment.

  Hey hey. Finally reviewing this. Sorry for the delay, @noarave. Here are a 
couple of comments:
  
  1. The current implementation applies the Codex link mixin styles to all <a> 
elements via app.scss. Nevertheless, this styling should be reserved only for 
text elements (or text+icon). The MSMF logo shouldn't display the link's 
outline when active/focused:
  
  F41600651: Screenshot 2023-12-14 at 17.41.10.png 
<https://phabricator.wikimedia.org/F41600651>
  
  2. The "Log in" link displays the color #a2a9b1 in its default state (the 
style of all other states are fine). It appears to be inheriting it from the 
auth-widget div:
  
  F41600687: Screenshot 2023-12-14 at 18.04.00.png 
<https://phabricator.wikimedia.org/F41600687>
  
  3. This is rather an improvement, so let's evaluate if we should capture it 
in a separate small task instead (att @Arian_Bozorg): It's not relevant for 
users to know if they have visited the "Log in", "Log out" or their user page 
(involuntary revisits are unlikely). The auth links don't need to display a 
visited state.
  
  F41600698: Screenshot 2023-12-14 at 18.15.57.png 
<https://phabricator.wikimedia.org/F41600698>

TASK DETAIL
  https://phabricator.wikimedia.org/T347161

EMAIL PREFERENCES
  https://phabricator.wikimedia.org/settings/panel/emailpreferences/

To: noarave, Sarai-WMDE
Cc: Arian_Bozorg, ItamarWMDE, Aklapper, Michael, Sarai-WMDE, 
Danny_Benjafield_WMDE, Astuthiodit_1, STH, karapayneWMDE, Invadibot, 
maantietaja, Mattia_Capozzi_WMDE, Akuckartz, Nandana, Lahi, Gq86, 
GoranSMilovanovic, QZanden, LawExplorer, _jensen, rosalieper, Scott_WUaS, 
Volker_E, Wikidata-bugs, aude, Lydia_Pintscher, Mbch331
_______________________________________________
Wikidata-bugs mailing list -- wikidata-bugs@lists.wikimedia.org
To unsubscribe send an email to wikidata-bugs-le...@lists.wikimedia.org

Reply via email to