Closed
Bug 1548094
Opened 6 years ago
Closed 6 years ago
remove grid usage from comm/mailnews/extensions/newsblog/content/feed-subscriptions.xul
Categories
(Thunderbird :: General, task)
Thunderbird
General
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 68.0
People
(Reporter: khushil324, Assigned: khushil324)
References
Details
Attachments
(1 file, 1 obsolete file)
|
8.69 KB,
patch
|
khushil324
:
review+
|
Details | Diff | Splinter Review |
No description provided.
|
Assignee | |
Updated•6 years ago
|
Assignee: nobody → khushil324
Status: NEW → ASSIGNED
|
|
Assignee | |
Comment 1•6 years ago
|
||
Attachment #9061733 -
Flags: review?(mkmelin+mozilla)
|
|
Assignee | |
Updated•6 years ago
|
Blocks: tb-burn-xul-grids
|
||
Comment 2•6 years ago
|
||
Comment on attachment 9061733 [details] [diff] [review]
Bug-1548094_remove-grid-feed-subscriptions.patch
Review of attachment 9061733 [details] [diff] [review]:
-----------------------------------------------------------------
::: mailnews/extensions/newsblog/content/feed-subscriptions.xul
@@ +102,5 @@
> + </hbox>
> + </vbox>
> + <vbox flex="1">
> + <textbox id="nameValue"
> + oninput="FeedSubscriptions.setPrefs(this)"/>
while touching this, you could add the missing ; for a bunch of these. Like FeedSubscriptions.setPrefs(this);
Attachment #9061733 -
Flags: review?(mkmelin+mozilla) → review+
|
|
Assignee | |
Comment 3•6 years ago
|
||
Attachment #9061733 -
Attachment is obsolete: true
Attachment #9062191 -
Flags: review+
|
|
Assignee | |
Updated•6 years ago
|
Keywords: checkin-needed
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/427525170a6c
remove grid usage from feed-subscriptions.xul. r=mkmelin
|
||
Updated•6 years ago
|
Target Milestone: --- → Thunderbird 68.0
You need to log in
before you can comment on or make changes to this bug.
Description
•