The star panel should not replace the header with itself each time it's opened

RESOLVED FIXED in Firefox 13

Status

()

defect
RESOLVED FIXED
8 years ago
7 years ago

People

(Reporter: alice0775, Assigned: mak)

Tracking

(Depends on 1 bug, {regression})

Trunk
Firefox 13
x86
All
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

Reporter

Description

8 years ago
Build Identifier:
http://hg.mozilla.org/mozilla-central/rev/d71dab82fff4
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:13.0a1) Gecko/20120211 Firefox/13.0a1 ID:20120211031145

Header label of "Edit bookmarks panel" does not change till restart browser.
The header label should be "Page Bookmarked" or "Edit This Bookmark" depending on state of bookmark.

Reproducible: Always

Steps to Reproduce#1:
1. Start Firefox with new profile
2. Open non bookmarked page
3. CTRL+D  ---- you see "Page Bookmarked" as expected
4, Double Click STAR UI

Actual Results:
  The header label is "Page Bookmarked"

Expected Results:
  The header label should be "Edit This Bookmark"


Steps to Reproduce#2:
1. Start Firefox with new profile
2. Open a bookmarked page
3. Double Click STAR UI  ---- you see "Edit This Bookmark" as expected
4. Open non bookmarked page
5. CTRL+D


Actual Results:
  The header label is "Edit This Bookmark"

Expected Results:
  The header label should be "Page Bookmarked"

Regression window(cachedm-c)
Works:
http://hg.mozilla.org/mozilla-central/rev/7aa58a99a2e5
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:7.0a1) Gecko/20110630 Firefox/7.0a1 ID:20110630003937
Fails:
http://hg.mozilla.org/mozilla-central/rev/5c246f2bccb1
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:7.0a1) Gecko/20110630 Firefox/7.0a1 ID:20110630035829
Pushlog:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=7aa58a99a2e5&tochange=5c246f2bccb1


Regression window(cached m-i)
Works:
http://hg.mozilla.org/integration/mozilla-inbound/rev/6777320f6f29
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:7.0a1) Gecko/20110629 Firefox/7.0a1 ID:20110629215351
Fails:
http://hg.mozilla.org/integration/mozilla-inbound/rev/c2f48684b9f5
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:7.0a1) Gecko/20110629 Firefox/7.0a1 ID:20110629231921
Pushlog:
http://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=6777320f6f29&tochange=c2f48684b9f5


In local build:
first bad changeset c2f48684b9f5
last good changeset fb03584dd823
Reporter

Updated

8 years ago
See Also: → 725784
Reporter

Comment 1

8 years ago
Posted patch possible fix (obsolete) — Splinter Review
using setAttribute to set value strings
Reporter

Updated

8 years ago
OS: Windows 7 → All
Assignee

Updated

7 years ago
Depends on: 725784
Assignee

Updated

7 years ago
Assignee: nobody → mak77
Status: NEW → ASSIGNED
Component: DOM → Bookmarks & History
Product: Core → Firefox
QA Contact: general → bookmarks
Assignee

Updated

7 years ago
Summary: Header label of "Edit bookmarks panel" does not change till restart browser → The star panel should not replace the header with itself each time it's opened
Assignee

Comment 2

7 years ago
Comment on attachment 596493 [details] [diff] [review]
possible fix

Will shortly attach a patch that avoid the self replacement, since it was a no-op but now it's not anymore. We don't need to pay that cost.

Btw, would like to thank you for finding these bugs and the problem.
Attachment #596493 - Attachment is obsolete: true
Assignee

Comment 3

7 years ago
Posted patch patch v1.0Splinter Review
So basically, instead of moving it each time the panel is opened, this moves it when the binding is lazy loaded on first opening.
Attachment #600035 - Flags: review?(dietrich)
Attachment #600035 - Flags: review?(dietrich) → review+
Assignee

Comment 5

7 years ago
https://hg.mozilla.org/mozilla-central/rev/4e2e861f136b
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Assignee

Updated

7 years ago
Blocks: 638785

Updated

7 years ago
Blocks: 763708
You need to log in before you can comment on or make changes to this bug.