The default bug view has changed. See this FAQ.

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

RESOLVED INCOMPLETE

Status

Other Applications Graveyard
Nightly Tester Tools
--
enhancement
RESOLVED INCOMPLETE
5 years ago
5 years ago

People

(Reporter: xabolcs, Assigned: xabolcs)

Tracking

Dependency tree / graph

Details

Attachments

(1 attachment, 2 obsolete attachments)

(Assignee)

Description

5 years ago
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()
(Assignee)

Updated

5 years ago
Severity: normal → enhancement
Depends on: 682996

Comment 1

5 years ago
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

5 years ago
I would appreciate this very much. +1
(Assignee)

Comment 3

5 years ago
Created attachment 599408 [details]
Nightly Tester Tools with v1 patch for early adopters

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

5 years ago
Created attachment 599416 [details] [diff] [review]
add support of Panorama group name v1

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

5 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

5 years ago
Assignee: nobody → xabolcs
(Assignee)

Updated

5 years ago
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)
Depends on: 624394
(Assignee)

Comment 7

5 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

5 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

5 years ago
Created attachment 609416 [details]
Nightly Tester Tools with v4 patch for early adopters

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

5 years ago
Resolving as INCOMPLETE due to the Bugzilla Component cleanup.
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → INCOMPLETE

Comment 11

5 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?
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.