Last modified: 2014-03-13 19:58:31 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T61636, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 59636 - Flow: should not override global h1 and h2 style
Flow: should not override global h1 and h2 style
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
Flow (Other open bugs)
master
All All
: Normal normal (vote)
: ---
Assigned To: Nobody - You can work on this!
: design
: 62593 (view as bug list)
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-01-04 04:16 UTC by spage
Modified: 2014-03-13 19:58 UTC (History)
6 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments
hidden post's titlebar when using the Typography refresh Beta feature (26.63 KB, image/png)
2014-02-07 07:32 UTC, spage
Details

Description spage 2014-01-04 04:16:50 UTC
Flow boards set the h1 page heading to font-family: Georgia, color: #1E1E1E, with no bottom border. They also set h2 outside topics to color: #231f20 with no bottom border. But then the Typography refresh Beta feature sets both h1 and h2 to font-family: "DejaVu Serif", Georgia, serif.  It's confusing: it suggests titles of Flow boards will appear different from regular pages when the font change will be minimal when Vector gets the Typography refresh; templates in a Flow board header appear differently than on regular pages; if you use a Flow board to discuss the Typography refresh you get slightly different fonts.

If we think changing header typography is a generally good idea, we should do it globally in Vector; if we think Flow topics and the Flow header need different typography than the default we should give them targeted selectors (as modules/discussion/styles/topic.less already does for the topic title) rather than redefining generic tags, and be sure to say why in the comments. Dropping the bottom border from h1 is an aesthetic choice that won't work when we use a <div id="contentSub"> under the heading line to improve navigation back to a board.

I think Matthias added global h2 styling for "Historical view of a topic"; if we really need this it could be a specific .flow-history-log h2 selector.

See also bug 59155.
Comment 1 Bingle 2014-01-04 04:42:24 UTC
The WMF core features team tracks this bug on Mingle card https://mingle.corp.wikimedia.org/projects/flow/cards/684, but people from the community are welcome to contribute here and in Gerrit.
Comment 2 Gerrit Notification Bot 2014-01-04 04:49:52 UTC
Change 105426 had a related patch set uploaded by Spage:
Don't globally style h1 and h2

https://gerrit.wikimedia.org/r/105426
Comment 3 spage 2014-02-07 07:32:54 UTC
Created attachment 14519 [details]
hidden post's titlebar when using the Typography refresh Beta feature
Comment 4 spage 2014-02-07 07:33:31 UTC
On mediawiki.org, go to Special:Preferences > Beta > Typography refresh then visit a Flow page that has a hidden post. The serif h2 header typography affects hidden posts. See attached. I haven't tried with the patch.

May Galloway specified that h2 in topic titles should continue to have the current sans serif layout.
Comment 5 Gerrit Notification Bot 2014-03-03 12:36:58 UTC
Change 116474 had a related patch set uploaded by Matthias Mullie:
(bug 59636) should not override global h1 and h2 style

https://gerrit.wikimedia.org/r/116474
Comment 6 Maryana Pinchuk 2014-03-13 00:21:58 UTC
*** Bug 62593 has been marked as a duplicate of this bug. ***
Comment 7 Gerrit Notification Bot 2014-03-13 17:37:27 UTC
Change 116474 merged by jenkins-bot:
(bug 59636) should not override global h1 and h2 style

https://gerrit.wikimedia.org/r/116474
Comment 8 Gerrit Notification Bot 2014-03-13 19:58:31 UTC
Change 105426 abandoned by EBernhardson:
Don't globally style h1 and h2

Reason:
116474 was just merged which solves the same problem

https://gerrit.wikimedia.org/r/105426

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links