Closed Bug 549523 Opened 15 years ago Closed 15 years ago

Weave is doing weird things in the tabs menu

Categories

(Firefox :: Sync, defect)

Product:
Firefox
Component:
Sync
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED WONTFIX

People

(Reporter: zpao, Unassigned)

Details

After hearing Dietrich mention that this were a bit weird for him (in comparison to recently closed tabs/windows)... * We're not setting "bookmark-item" on the class * Why are we doing item.style.listItemImage? We can (and likely should) set the image attribute * We should use the favicon service if we have the icon there. * Is there a reason to not set oncommand on each item? I don't see a reason to do it, but it jumped out at me. * Maybe some other things, but I guess not important.
bug 539056 is about killing this menu, it's really non-awesome UI, and doesn't scale to people's (*cough*Blair*cough*) tabsets.
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → WONTFIX
Component: Firefox Sync: Backend → Sync
Product: Cloud Services → Firefox
You need to log in before you can comment on or make changes to this bug.