https://bugzilla.wikimedia.org/show_bug.cgi?id=60239
--- Comment #9 from Jared Zimmerman (WMF) ---
This looks fine, thanks, but I want to reiterate…
Please upload screenshots of any proposed (skin) changes PRIOR to integrating
them.
for visual things a visual review is just as important as a co
https://bugzilla.wikimedia.org/show_bug.cgi?id=60239
Oliver Keyes changed:
What|Removed |Added
Status|PATCH_TO_REVIEW |RESOLVED
Resolution|---
https://bugzilla.wikimedia.org/show_bug.cgi?id=60239
--- Comment #8 from Gerrit Notification Bot ---
Change 108512 merged by jenkins-bot:
Changed the font size used for subheading in notificatons
https://gerrit.wikimedia.org/r/108512
--
You are receiving this mail because:
You are on the CC li
https://bugzilla.wikimedia.org/show_bug.cgi?id=60239
--- Comment #7 from Oliver Keyes ---
11px LGTM.
--
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.w
https://bugzilla.wikimedia.org/show_bug.cgi?id=60239
--- Comment #6 from Kunal Mehta (Legoktm) ---
Created attachment 14362
--> https://bugzilla.wikimedia.org/attachment.cgi?id=14362&action=edit
11px screenshot
Screenshot of https://gerrit.wikimedia.org/r/#/c/108512/2 applied on enwiki
--
Yo
https://bugzilla.wikimedia.org/show_bug.cgi?id=60239
--- Comment #5 from Jared Zimmerman (WMF) ---
The rational for the smaller font size was to clearly differentiate between
primary and secondary actions, I can see the secondary action text being
slightly larger but I think there should be a cle
https://bugzilla.wikimedia.org/show_bug.cgi?id=60239
--- Comment #4 from Erwin Dokter ---
14px is is *bigger* then the base font size, which is 13px. Please change to
11px, the value I suggested in
https://en.wikipedia.org/wiki/Wikipedia_talk:Notifications#Font_size
--
You are receiving this ma
https://bugzilla.wikimedia.org/show_bug.cgi?id=60239
--- Comment #3 from Andre Klapper ---
roopak: Why this is size (especially compared to other sizes)? Please
elaborate.
--
You are receiving this mail because:
You are on the CC list for the bug.
___
https://bugzilla.wikimedia.org/show_bug.cgi?id=60239
Gerrit Notification Bot changed:
What|Removed |Added
Status|NEW |PATCH_TO_REVIEW
--
You are
https://bugzilla.wikimedia.org/show_bug.cgi?id=60239
--- Comment #2 from Gerrit Notification Bot ---
Change 108512 had a related patch set uploaded by Roopak:
Changed the font size used for subheading in notificatons
https://gerrit.wikimedia.org/r/108512
--
You are receiving this mail because:
https://bugzilla.wikimedia.org/show_bug.cgi?id=60239
Tony Thomas <01tonytho...@gmail.com> changed:
What|Removed |Added
CC||01tonytho...@gmai
https://bugzilla.wikimedia.org/show_bug.cgi?id=60239
roopak changed:
What|Removed |Added
CC||mailsto...@gmail.com
--- Comment #1 from roop
12 matches
Mail list logo