1336434 - "Your Top Sites" should say "Top Sites" when we're using the default set

Status: RESOLVED FIXED

(Firefox :: New Tab Page, defect)

Description

[:Gijs (he/him)]

Per the qx meeting notes:


* question for verdi: should the heading read "Top Sites" initially when there are no history tiles?

-->  Yes - no history or default sites it should say "Top Sites" and when your stuff is imported it should say "Your top sites"

Comment 1

[Anjul Tyagi]

I am working on this.

Comment 2

[Anjul Tyagi]

I've changed the string in the newTab.dtd file from "Show your top sites" to "Your top sites". Is this all?

Comment 3

[Anjul Tyagi]

I've changed the string in the newTab.dtd file from "Show your top sites" to "Top sites". Is this all ?

Comment 4

[Anjul Tyagi]

If we can just replace "your top sites" with "top sites", it should be a more generic solution to this.

Comment 5

[:Gijs (he/him)]

Hi Anjul,

I'm afraid this isn't a mentored bug, and also isn't the easiest to work on. It's related to a new feature that is not enabled by default. Perhaps take a look at https://www.joshmatthews.net/bugsahoy/?ff=1&unowned=1 to see if there are other bugs there that you'd be interested in working on?

Comment 6

[Dão Gottwald [:dao]]

Attachment: Bug 1336434 - "Your Top Sites" heading should say "Top Sites" instead when using the default set.

Review commit: https://reviewboard.mozilla.org/r/112348/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/112348/

Comment 7

[:Gijs (he/him)]

Comment on attachment 8837108 [details]
Bug 1336434 - "Your Top Sites" heading should say "Top Sites" instead when using the default set.

https://reviewboard.mozilla.org/r/112348/#review113646

Tests for this might be a good idea?

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/d65e47dee9a4
"Your Top Sites" heading should say "Top Sites" instead when using the default set. r=Gijs

Comment 9

[Dão Gottwald [:dao]]

(In reply to :Gijs from comment #7)
> Comment on attachment 8837108 [details]
> Bug 1336434 - "Your Top Sites" heading should say "Top Sites" instead when
> using the default set.
> 
> https://reviewboard.mozilla.org/r/112348/#review113646
> 
> Tests for this might be a good idea?

I'd rather not bake en-US specifics into tests. Also, given the plan to have Activity Stream code replace this implementation, it seems that this won't ever ship to end users beyond a funnelcake.

Comment 10

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/d65e47dee9a4

Comment 11

[Dão Gottwald [:dao]]

Attachment: patch for uplift

Approval Request Comment
[Feature/Bug causing the regression]: bug 1334444 / bug 1332318
[User impact if declined]: see comment 0
[Is this code covered by automated tests?]: no
[Has the fix been verified in Nightly?]: no
[Needs manual test from QE? If yes, steps to reproduce]: no
[List of other uplifts needed for the feature/fix]: /
[Is the change risky?]: no
[Why is the change risky/not risky?]: it's limited to browser.newtabpage.compact = true (not the default)
[String changes made/needed]: string hardcoded for the funnelcake, no l10n needed

Comment 12

[Dão Gottwald [:dao]]

Attachment: patch for uplift

see previous comment

Comment 13

[Julien Cristau [:jcristau]]

Comment on attachment 8837717 [details] [diff] [review]
patch for uplift

about:newtab changes for funnelcake, aurora53+, beta52+

Comment 14

[Ryan VanderMeulen (PTO, back July 27)]

https://hg.mozilla.org/releases/mozilla-aurora/rev/5b366c35d4a5

Comment 15

[Ryan VanderMeulen (PTO, back July 27)]

https://hg.mozilla.org/releases/mozilla-beta/rev/7c6a0bf51125

Comment 16

[Ryan VanderMeulen (PTO, back July 27)]

https://hg.mozilla.org/releases/mozilla-esr52/rev/7c6a0bf51125

Comment 17

[Saheda Reza [:Antora]]

I have reproduced this according to Firefox nightly (2017-02-03)

Fixing bug is verified on Latest Developer Edition, Latest Beta , Latest Nightly

 -- Build ID:(20170303004003), User Agent: Mozilla/5.0 (Windows NT 10.0; rv:53.0) Gecko/20100101 Firefox/53.0
 -- Build ID:(20170307064827),User Agent:Mozilla/5.0 (Windows NT 10.0; rv:53.0) Gecko/20100101 Firefox/53.0
 -- Build ID:(20170308030207), User Agent:Mozilla/5.0 (Windows NT 10.0; rv:55.0) Gecko/20100101 Firefox/55.0

Tested OS-- Windows10 32bit
[bugday-20170308]