Closed Bug 1309692 Opened 8 years ago Closed 8 years ago

Unique page title is missing on the refreshed Foundation pages

Categories

(www.mozilla.org :: Pages & Content, defect)

Production
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: kohei, Assigned: agibson)

References

Details

(Keywords: regression)

Attachments

(2 files)

44 bytes, text/x-github-pull-request
Details | Review
44 bytes, text/x-github-pull-request
Details | Review
+++ This bug was initially created as a clone of Bug #1307107 +++

I guess this is a regression from Bug 1307107. The <h1> on the all(?) pages under /foundation/ is now "The Mozilla Foundation" so it's difficult to recognize each page's topic.

For example,
https://www.mozilla.org/en-US/foundation/licensing/binary-components/
The <h1> should be Mozilla Licensing Policies

https://www.mozilla.org/en-US/foundation/documents/
The <h1> should be Mozilla Foundation Documents

and so on.
Flags: needinfo?(alanm)
(In reply to Kohei Yoshino [:kohei] from comment #0)
> https://www.mozilla.org/en-US/foundation/licensing/binary-components/
> The <h1> should be Mozilla Licensing Policies

In this case, actually <h2> is missing. <h1> could be "Mozilla Licensing Policies - Binary Components"
Attached file pull request
Assignee: nobody → kohei.yoshino
Status: NEW → ASSIGNED
Flags: needinfo?(alanm)
I think the appropriate solution here is to add `<h2>` elements to pages where they're missing. Doing that will

* keep the cascade of headings in the appropriate order across the pages
* remain consistent with the stakeholder and designer vision
* not create very thick red bars across the top of the pages

I also feel like the shortened `<title>` elements are fine; as they get longer they'll just get truncated in many places anyway.
Bobby - thinking your input is appropriate here?
Flags: needinfo?(bobby)
Yeah, I think that's wise. Let me summon Sabrina to make sure.
Flags: needinfo?(bobby) → needinfo?(sabrina)
For the design refresh, the main page along with three additional "foundation program" subpages were the focus so those designs are complete (includes h1 and h2). Now seeing how other old pages are applying the new style too, it doesn’t have the corresponding heading taken into consideration to describe the page so Kohei and Alan’s have good suggestions to include it back so it isn't just "The Mozilla Foundation" that describes the page on top.
Flags: needinfo?(sabrina)
Friendly ping. Any further discussion needed here? I think we've reached consensus that we should add second-level headings to each page.
Sounds good to me. Thanks for following up on gh.
Assignee: kohei.yoshino → nobody
Status: ASSIGNED → NEW
Assignee: nobody → agibson
Status: NEW → ASSIGNED
Attached file GitHub pull request
Commits pushed to master at https://github.com/mozilla/bedrock

https://github.com/mozilla/bedrock/commit/47f23f0ab3fc8a838cd14cb41d7c79417fd1c452
[fix bug 1309692] Unique page title is missing on the refreshed Foundation pages

https://github.com/mozilla/bedrock/commit/668f6e95a88d804980c1bc01f441282a410cc364
Unify foundation page titles and headings

[fix bug 1309692]

https://github.com/mozilla/bedrock/commit/3f6bdeb78f97776291f6db13eca5c4001696f6e6
Merge pull request #4576 from alanmoo/alanmoo-bug-1309692-foundation-page-titles

[fix bug 1309692] Clean up and foundation page titles and headers
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: