Closed Bug 726560 Opened 14 years ago Closed 13 years ago

Add support of Panorama group name (FF10+) for titlebar customization

Categories

(Other Applications Graveyard :: Nightly Tester Tools, enhancement)

enhancement
Not set
normal

Tracking

(Not tracked)

RESOLVED INCOMPLETE

People

(Reporter: xabolcs, Assigned: xabolcs)

References

Details

Attachments

(1 file, 2 obsolete files)

User Agent: Mozilla/5.0 (Windows NT 5.1; rv:12.0a2) Gecko/20120212 Firefox/12.0a2 Build ID: 20120212042014 Steps to reproduce: Displaying of the active group name in the titlebar was discounted by Bug 682996 which is landed in FF10. Enhancing NTT's titlebar customizing template with such an element, users could have the ability to "revert" the removing of TabView.getActiveGroupName()
Severity: normal → enhancement
Depends on: 682996
I second this. I think it would be good to have an "about:config" option at least to display the group name in the title bar. It is quite important for me.
I would appreciate this very much. +1
Attaching an installable package of NTT with a work-in-progress patch. Dear dinesy and John: could You test this patched NTT?
In this v1 patch I: - added a lot stuff to nightlyApp, heavily based on Bug 682996's changeset: a constant, a getter, a listener and a little initializing code - added a new variable with ActiveTabGroupName, and it's related strings - moved nightly.init's main code to run delayed: nightly.initLazy, because NTT gets initialized earlier than TabView and therefore the variable remains empty Please help me! I have got some question while implementing this patch. What about the comments which originates from TabView? What about the new names? What about the places of the insertions? Thank You for Your answers! And the r? - freely after Tony. :) "Heather, Henrik, whoever of you two comes around first to review the patch may remove the other. Or of course, if you think that some other reviewer would be better qualified, feel free to change the reviewer's name."
Attachment #599416 - Flags: review?(hskupin)
Attachment #599416 - Flags: review?(fayearthur)
(In reply to Szabolcs Hubai from comment #4) > Created attachment 599416 [details] [diff] [review] > [...] > because NTT gets initialized earlier than TabView and therefore the > variable remains empty > [...] s/TabView/SessionStore/
Assignee: nobody → xabolcs
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Thanks Szabolcs for starting to work on a patch. Would you mind to transfer the patch over to a github pull request? https://github.com/mozilla/nightlytt/pulls We are considering to get rid of the Bugzilla component in favor of handling everything directly via github. I would be happy to have a look at your patch as soon as it will be available as pull request.
Attachment #599416 - Flags: review?(hskupin)
Attachment #599416 - Flags: review?(fayearthur)
Hi Henrik, OK, I'll transfer them to GitHub. In the meantime would You mind to create a wiki page (on MozWiki or even on GitHub) about the GitHub routine? See Tony's comment in Bug 715557 Comment 1 and #C9! Because I really like that Bugzilla filters out the pollution generated by the reviewing system. In other words: I like the one commit - one issue/bug commit history. :) (Of course: part 1, part 2, part ... are welcome) In other words, again: I don't like the commits like "assigned nits". :P
Comment on attachment 599416 [details] [diff] [review] add support of Panorama group name v1 At GitHub there is a more recent patch. This is obsolete now.
Attachment #599416 - Attachment is obsolete: true
I saw noticeable activity in Bug 682996, therefore I packed up an NTT version which contains my recent work up to commit e11907e1 [1]. Please feel free to comment / report problems with it at Github Issue 19! [2] [1] - https://github.com/xabolcs/nightlytt/commit/e11907e1b0e5cfeccffb7013cf42c27671a3b561 [2] - https://github.com/mozilla/nightlytt/pull/19
Attachment #599408 - Attachment is obsolete: true
Resolving as INCOMPLETE due to the Bugzilla Component cleanup.
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → INCOMPLETE
I'm confused. This is still a bug regardless of the component status. Is this just a clever way of sweeping this under the rug?
Ravi, we are in the process to move all the existing and open bugs over to Github where we can track the whole development efforts on a single place as forcing users to learn two different systems. So if you are still interested in this bug feel free to watch the activity on the Github repository. The existing NTT component on Bugzilla will be abandoned soon.
Product: Other Applications → Other Applications Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: