1298615 - Adjust output of 'Insert Code Sample Template' button

Status: VERIFIED FIXED

(developer.mozilla.org Graveyard :: Editing, enhancement)

Description

[Sebastian Zartner [:sebo]]

+++ This bug was initially created as a clone of Bug #1246684 +++

Two things should be done:
1. Lowercase all occurrences of the word 'Content'[1]
2. Remove the 'name' attribute from the heading[2]
3. Remove the spaces from the macro[3]

Sebastian

[1] https://developer.mozilla.org/en-US/docs/MDN/Contribute/Guidelines/Writing_style_guide#Page_titles

[2] https://github.com/Elchi3/mdn-doc-tests/blob/master/data/tests/name-attribute.js (not an official rule, but marked as error by the MDN documentation tester add-on)

[3] https://developer.mozilla.org/en-US/docs/MDN/Contribute/Structures/Macros#Using_a_macro_in_content (not an official rule, though in the examples macros never have spaces.

Comment 1

[Eric Shepherd [:sheppy]]

Yes please.

Although I thought we'd decided to use headings "JavaScript", "HTML", and "CSS" rather than "JavaScript content" etc. I could be mistaken though. Doesn't matter a lot, I suppose.

Comment 2

[Sebastian Zartner [:sebo]]

(In reply to Eric Shepherd [:sheppy] from comment #1)
> Yes please.
> 
> Although I thought we'd decided to use headings "JavaScript", "HTML", and
> "CSS" rather than "JavaScript content" etc. I could be mistaken though.
> Doesn't matter a lot, I suppose.

You're right. At least that's what I get from https://groups.google.com/d/topic/mozilla.dev.mdc/49oqJAHFnWQ/discussion.

Sebastian

Comment 4

[Eric Shepherd [:sheppy]]

I've submitted https://github.com/mozilla/kuma/pull/4190, which fixes this issue.

Comment 5

[[github robot]]

Commits pushed to master at https://github.com/mozilla/kuma

https://github.com/mozilla/kuma/commit/a73a6a23637b15739d5d6c43496d8035be8a8e52
Bug 1298615 - Fix output of Insert Live Sample Template button

This fixes the issues in bug 1298615 as follows:

1. The labels of the subsections are now simply "CSS", "HTML", and
"JavaScript". This affects translations although the changes should
be pretty minimal.

2. Removed the "name" attribute from the headings.

3. Removed the unneeded spaces from inside the generated EmbedLiveSample
macro.

https://github.com/mozilla/kuma/commit/9601018a62559cfbf6ee4826301a3295bb3f2280
Merge pull request #4190 from mozilla/fix-livesample-subtitles-1298615

Bug 1298615 - Fix output of Insert Live Sample Template button

Comment 6

[John Whitlock [:jwhitlock]]

Deployed to staging and production

Comment 7

[Sebastian Zartner [:sebo]]

Works as expected. Thanks!
The only missing part is that "Content" should also have been turned into lowercase in the sample texts, but that's not a big deal as they will be replaced by the people, anyway.

Sebastian