When we implemented the pref UI for managing your tags, we didn't get around to implementing the ability to Edit Tags. You should be able to change the name and the color of a tag.
Seamonkey is under a string freeze on the branch so they may not be able to benefit from this patch on the branch.
Status: NEW → ASSIGNED
Created attachment 247236 [details] [diff] [review] the fix 1) If you pass in a tag key as an argument into the new tag dialog, we'll turn the dialog into an edit dialog. 2) If you try to change the name of a tag to a name of a tag that already exists, you get prompted with an alert. 3) I changed the behavior of nsMsgTagService::GetKeyForTag to not throw an error when a key doesn't exist for a tag name. This made the front end code that actually uses this routine easier since we don't have to wrap things in try/catch clauses when calling getKeyForTag.
Comment on attachment 247236 [details] [diff] [review] the fix Neil or Karsten, do you mind reviewing the mailnews change to newTagDialog.js? For seamonkey to leverage this on the trunk, we just need to pass in the key for the tag being edited as a window argument (along with the string change to messenger.properties)
> Neil or Karsten, do you mind reviewing the mailnews change to newTagDialog.js? Since bug 342560 is fixed on trunk and branch, SeaMonkey does not use newTagDialog.js/xul anymore. :)
Comment on attachment 247236 [details] [diff] [review] the fix I don't think we use this dialog any more.
I was just noticing the seamonkey edit tags preferences UI. Very nice!
Comment on attachment 247236 [details] [diff] [review] the fix David, see comment 2 for details.
Attachment #247236 - Flags: superreview?(bienvenu)
Attachment #247236 - Flags: superreview?(bienvenu) → superreview+
Status: ASSIGNED → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird2.0
Created attachment 247265 [details] [diff] [review] As per comment #2 part 3)
Attachment #247265 - Flags: review?(mnyromyr)
(In reply to comment #6) > I was just noticing the seamonkey edit tags preferences UI. Very nice! Scott, do you plan to add the equivalent of SM's Raise/Lower Importance for TB? I can verify your current patch works, but I didn't know if this was actually complete. Nit: I would have used "Precedence" rather than "Importance."
Created attachment 247636 [details] [diff] [review] forgot to checkin this change I just noticed this diff sitting in my tree that I forgot to check in when I landed edit tag functionality. This is going to really suck because it means you can't add a new tag in beta 1 :(
I just successfully tested changing the text and color of a tag with TB 2 B2 (20070116), thank you very much! But I recognized that the change is not applied to the current view immediately. 1) I sorted the tag column and changed just the tag which was now displayed for all the first mails. 2) The change was only visible after actively changing the view, i.e. by scrolling or resorting.
You mean it doesn't resort immediately on adding the tag? That's intended. See bug 119162.
Resorting or removing the current message because of an activated filter after the label change was not my intention. I was just wondering why the already displayed mails with the old label does not get updated with the new label (text and color).
> I was just wondering why the already displayed mails with the old label does > not get updated with the new label (text and color). The tag list shows the tags TB should know about. All other tags on messages are ignored. Changing a tag in the prefs just tells TB to forget about that tag, it does not change any tag on any message - the old tag will still be there, just hidden.
I just can tell what I saw ;) 1) Sort the current view to show many messages containing one existing tag. 2) Open prefs and change the name and/or color of that particular tag. 3) Current view is not updated until another action has been occured (scrolling, sorting, etc.). 4) All mails previously contained the changed tag are showing the new tag now.
Any chance of providing tag reordering/renumbering in the tag preferences interface? In the current system, there's no method other than deletion for moving tags off the number keys after I've added them. I'm using tagging for projects and other concerns, but some tags will be less important over time (I'll still want to read/search on them, but much less likely to add them). For this reason, the ability to reorder tags would make the whole tag feature much more useful.
Yoz, I've just opened bug 369114 for that feature.
verified for 22.214.171.124 using Mozilla/5.0 (Windows; U; Windows NT 5.2; en-US; rv:126.96.36.199) Gecko/20070326 Thunderbird/188.8.131.52 Mnenhy/0.7.5.0 ID:2007032620 (Thunderbird 2 RC1) - Editing Tags works fine
Keywords: fixed184.108.40.206 → verified220.127.116.11
You need to log in before you can comment on or make changes to this bug.