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)
Other Applications Graveyard
Nightly Tester Tools
Tracking
(Not tracked)
RESOLVED
INCOMPLETE
People
(Reporter: xabolcs, Assigned: xabolcs)
References
Details
Attachments
(1 file, 2 obsolete files)
132.20 KB,
application/x-xpinstall
|
Details |
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()
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.
Comment 2•14 years ago
|
||
I would appreciate this very much. +1
Assignee | ||
Comment 3•14 years ago
|
||
Attaching an installable package of NTT with a work-in-progress patch.
Dear dinesy and John: could You test this patched NTT?
Assignee | ||
Comment 4•14 years ago
|
||
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)
Assignee | ||
Comment 5•14 years ago
|
||
(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 | ||
Updated•14 years ago
|
Assignee: nobody → xabolcs
Assignee | ||
Updated•14 years ago
|
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Comment 6•14 years ago
|
||
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.
Updated•14 years ago
|
Attachment #599416 -
Flags: review?(hskupin)
Attachment #599416 -
Flags: review?(fayearthur)
Assignee | ||
Comment 7•14 years ago
|
||
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
Assignee | ||
Comment 8•13 years ago
|
||
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
Assignee | ||
Comment 9•13 years ago
|
||
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
Assignee | ||
Comment 10•13 years ago
|
||
Resolving as INCOMPLETE due to the Bugzilla Component cleanup.
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → INCOMPLETE
Comment 11•13 years ago
|
||
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?
Comment 12•13 years ago
|
||
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.
Updated•13 years ago
|
Product: Other Applications → Other Applications Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•