Rename tab.tabItem

RESOLVED FIXED in Firefox 4.0b10

Status

Firefox Graveyard
Panorama
RESOLVED FIXED
8 years ago
2 years ago

People

(Reporter: iangilman, Assigned: raymondlee)

Tracking

({dev-doc-needed})

unspecified
Firefox 4.0b10
dev-doc-needed

Details

(Whiteboard: [cleanup][qa-][API])

Attachments

(1 attachment, 2 obsolete attachments)

(Reporter)

Description

8 years ago
We hang a TabItem off of xul:tabs, and currently we do it with the property "tabItem", but this is too generic for the context. I propose we change it to "_tabViewTabItem" to identify what it is, where it's from, and that it's not for general consumption.
If we're going to take an opportunity to rename things, though, why not _panoramaTabItem? Most code references are to TabView, now, but that may get confusing in the long run... might as well start to switch over now?
Whiteboard: [cleanup][qa-]
"tabview" seems alright as an internal name. I don't see us replacing all its occurrences with "panorama".
(Assignee)

Updated

8 years ago
Assignee: nobody → raymond
Raymond: ping?
(Assignee)

Comment 4

8 years ago
(In reply to comment #3)
> Raymond: ping?

Dao: I agree with keeping "tabview" and change tab.tabItem to tab._tabViewTAbItem.
Alright, are you going to write the patch?
(Assignee)

Comment 6

8 years ago
Not at the moment, since it's a not blocker.
This should be done asap (in fact, it shouldn't have landed this way), since the later it gets done the more extensions will be affected.
(Assignee)

Comment 8

8 years ago
You are right.  Let me work on this next then.
(Assignee)

Comment 9

8 years ago
Created attachment 502750 [details] [diff] [review]
v1
Attachment #502750 - Flags: review?(ian)
(Assignee)

Updated

8 years ago
Status: NEW → ASSIGNED
(Assignee)

Comment 10

8 years ago
(In reply to comment #9)
> Created attachment 502750 [details] [diff] [review]
> v1

Passed Try
(Reporter)

Comment 11

8 years ago
Comment on attachment 502750 [details] [diff] [review]
v1

Looks great!
Attachment #502750 - Flags: review?(ian) → review+
(Assignee)

Updated

8 years ago
Attachment #502750 - Flags: approval2.0?
(Assignee)

Comment 12

8 years ago
It would be nice to land this as soon as we can.  As mentioned in comment #7, the later it gets done the more extensions will be affected.
We've already told add-on authors that we're frozen; I'm fine taking this, but I would like to see a blog post explaining what's changed or some outreach to add-on developers.

Adding Jorge.
Attachment #502750 - Flags: approval2.0? → approval2.0+
(Assignee)

Comment 14

8 years ago
Created attachment 503063 [details] [diff] [review]
Patch for check-in
Attachment #502750 - Attachment is obsolete: true
(Assignee)

Comment 15

8 years ago
Jorge: Basically, we change a name of the reference from tab.tabItem to tab._tabViewTabItem.  Extensions can access the panorama data related to a browser browser using tab._tabViewTabItem instead of tab.tabItem.
Keywords: checkin-needed
(In reply to comment #15)
> Jorge: Basically, we change a name of the reference from tab.tabItem to
> tab._tabViewTabItem.  Extensions can access the panorama data related to a
> browser browser using tab._tabViewTabItem instead of tab.tabItem.

Though it should be noted that this is no blessed API.

We should probably add a TabView.getItemForTab method.
http://hg.mozilla.org/mozilla-central/rev/847b7c31b8a5
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 4.0b10
(In reply to comment #16)
> Though it should be noted that this is no blessed API.

Definitely. Work on "a real API" has been punted past fx4.
Whiteboard: [cleanup][qa-] → [cleanup][qa-][API]
(Assignee)

Comment 19

8 years ago
Created attachment 503396 [details] [diff] [review]
Patch for check-in

Updated tab.tabItem to tab._tabViewTabItem
Attachment #503063 - Attachment is obsolete: true
Comment on attachment 503396 [details] [diff] [review]
Patch for check-in

This doesn't seem to belong in this bug.
Attachment #503396 - Attachment is obsolete: true

Updated

8 years ago
Attachment #503063 - Attachment is obsolete: false
This might be worth a mention on the Firefox 4 MDC page.
Keywords: dev-doc-needed
Hm, this is not presently documented at all. Any design notes or anything for the panorama view from a developer standpoint I can use to add reference docs for this stuff?
(Reporter)

Comment 23

8 years ago
(In reply to comment #22)
> Hm, this is not presently documented at all. Any design notes or anything for
> the panorama view from a developer standpoint I can use to add reference docs
> for this stuff?

Unfortunately we never got around to making an official API, so everything that's in Panorama is subject to change in the future. That said, our code documentation is here:

http://hg.mozilla.org/labs/tabcandy/raw-file/tip/content/doc/index.html

Let me know if you have questions.
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.