Page title not set in about:url-classifier

RESOLVED FIXED in Firefox 59

Status

()

defect
P3
normal
RESOLVED FIXED
2 years ago
a year ago

People

(Reporter: francois, Assigned: steveisok)

Tracking

(Blocks 1 bug, {good-first-bug})

unspecified
mozilla59
Points:
---

Firefox Tracking Flags

(firefox59 fixed)

Details

Attachments

(1 attachment)

(Reporter)

Description

2 years ago
When I open about:url-classifier:

- the title of the window is just "Nightly", and 
- the title of the tab is the one from the page I was on before

Expected:

The title of both window and tab is "URL Classifier Information".
(Assignee)

Comment 1

a year ago
Figured this was as good of a first bug as any... 

Submitted a review and attached your name, but it says you're not accepting review requests.  Is there someone else I should choose as a reviewer?

Thanks!
(In reply to Steve Pfister from comment #1)
> Figured this was as good of a first bug as any... 
> 
> Submitted a review and attached your name, but it says you're not accepting
> review requests.  Is there someone else I should choose as a reviewer?
> 
> Thanks!

If you need a reviewer, you can add "r?dimi" in the end of patch description.
Thanks for help!
(Assignee)

Comment 4

a year ago
Let me know if there's anything else I need to do.  Thanks!

Comment 5

a year ago
mozreview-review
Comment on attachment 8939708 [details]
Bug 1374811: Added title tag and changed the title of the page to URL Classifier Information;

https://reviewboard.mozilla.org/r/209562/#review216088

Looks good to me, thanks!
Attachment #8939708 - Flags: review?(dlee) → review+
Hi Steve,
Please help change the bug description to r=dimi instead of r=francois, thanks.
Comment hidden (mozreview-request)
(Assignee)

Comment 8

a year ago
Not sure if I did it the right way, but I ammended a new commit message and pushed for review again.  Is that what I was supposed to do in this instance?

Thanks!
(Assignee)

Comment 9

a year ago
mozreview-review
Comment on attachment 8939708 [details]
Bug 1374811: Added title tag and changed the title of the page to URL Classifier Information;

https://reviewboard.mozilla.org/r/209562/#review216316
Attachment #8939708 - Flags: review+
(Assignee)

Comment 10

a year ago
Also - what's next?  Do we just flag this bug as needing a check-in?  If I read the docs correctly, I think that's what is needed, but not 100% sure.
(In reply to Steve Pfister from comment #10)
> Also - what's next?  Do we just flag this bug as needing a check-in?  If I
> read the docs correctly, I think that's what is needed, but not 100% sure.

Yes, if you are ready to land this patch, flag checkin-needed and Sheriff will help land the patch.

Although this patch doesn't change much code, we should still run 'try' to make sure this patch doesn't break anything before landing.
See Try Wiki[1], you can also use Mozreview Automation Tab -> Trigger a Try Build.

If everything looks good then you can flag checkin-needed.

[1] https://wiki.mozilla.org/ReleaseEngineering/TryServer
(Assignee)

Comment 12

a year ago
I submitted a request for level 1 access.  Can you vouch for me?

Here's the link to the request:

https://bugzilla.mozilla.org/show_bug.cgi?id=1428767

Thanks!
(Assignee)

Comment 13

a year ago
Ok - I ran it through the try server... Lots of options, I see.  Picked the best ones and it all worked.
(Assignee)

Comment 14

a year ago
I'm not able to add the checkin-needed flag.  Can you guys add it for me?
Hi Steve,
Do you use
Edit Bug -> 'Tracking' category -> 'keywords' field? (The same field with good-first-bug)  

Add 'checkin-needed' to that field.
(Assignee)

Comment 16

a year ago
Sorry to keep this going... Looks like I don't have permission to change the keywords.  Can you do that?  And should I open a ticket w/ some team to get better permissions?

Comment 17

a year ago
Pushed by archaeopteryx@coole-files.de:
https://hg.mozilla.org/integration/autoland/rev/abb5be3dbdc7
Added title tag and changed the title of the page to URL Classifier Information; r=dimi
Keywords: checkin-needed

Comment 18

a year ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/abb5be3dbdc7
Status: NEW → RESOLVED
Last Resolved: a year ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla59
Hi, I'm really sorry to do this to a first patch, but I'm going to ask for a back-out. 

When a string changes, the string ID needs to change as well.
https://developer.mozilla.org/en-US/docs/Mozilla/Localization/Localization_content_best_practices#Changing_existing_strings
Assignee: nobody → steveisok
Backed out changeset abb5be3dbdc7 (bug 1374811) for changing existing strings without new IDs a=backout

https://hg.mozilla.org/mozilla-central/rev/0c2c46f76180a7d170faebdc84c21ea619f8d784

https://hg.mozilla.org/mozilla-central/rev/abb5be3dbdc7
Flags: needinfo?(francesco.lodolo)
Status: RESOLVED → REOPENED
Flags: needinfo?(francesco.lodolo) → needinfo?(steveisok)
Resolution: FIXED → ---
Target Milestone: mozilla59 → ---
(Assignee)

Comment 21

a year ago
After reading the localization section, it says the key doesn't have to change if the purpose hasn't.  From what I see, the title is more clear and is being used in > 1 places.  I definitely could be wrong though. 

That said, do you guys have any suggestions on what to change the key to?
Flags: needinfo?(steveisok)
(In reply to Steve Pfister from comment #21)
> After reading the localization section, it says the key doesn't have to
> change if the purpose hasn't.  

There is no mention of purpose in the section linked.

> If you are changing a string such that its meaning has changed, you must
> update the entity or property name for the string to match the new meaning. If
> your changes are relevant only for English — for example, to correct a
> typographical error or to make letter case consistent — then there is generally
> no need to update the entity name.

> That said, do you guys have any suggestions on what to change the key to?

aboutUrlClassifier.mainTitle or just aboutUrlClassifier.title would work, aboutUrlClassifier.pageTitle2 if you really want to keep the same name.
Comment hidden (mozreview-request)
(Assignee)

Comment 24

a year ago
The key change was made and submitted for review.

Comment 25

a year ago
mozreview-review
Comment on attachment 8939708 [details]
Bug 1374811: Added title tag and changed the title of the page to URL Classifier Information;

https://reviewboard.mozilla.org/r/209562/#review218426

Thanks, this looks good.
Attachment #8939708 - Flags: review+

Comment 26

a year ago
Pushed by francesco.lodolo@mozillaitalia.org:
https://hg.mozilla.org/integration/autoland/rev/f40f5ac940c1
Added title tag and changed the title of the page to URL Classifier Information; r=dimi,flod
https://hg.mozilla.org/mozilla-central/rev/f40f5ac940c1
Status: REOPENED → RESOLVED
Last Resolved: a year agoa year ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla59
You need to log in before you can comment on or make changes to this bug.