Closed
Bug 387485
Opened 18 years ago
Closed 6 years ago
Tag Editor
Categories
(Firefox :: Bookmarks & History, defect)
Firefox
Bookmarks & History
Tracking
()
RESOLVED
INACTIVE
Tracking | Status | |
---|---|---|
blocking2.0 | --- | - |
People
(Reporter: asaf, Unassigned)
References
Details
(Whiteboard: [places-ui])
Attachments
(5 files, 3 obsolete files)
245.99 KB,
image/png
|
Details | |
24.30 KB,
patch
|
Details | Diff | Splinter Review | |
23.04 KB,
image/png
|
Details | |
22.17 KB,
patch
|
Details | Diff | Splinter Review | |
27.92 KB,
image/png
|
Details |
Flags: blocking-firefox3?
Reporter | ||
Comment 1•18 years ago
|
||
Comment 2•18 years ago
|
||
some notes from Alex on the tag editor:
I think we should start by trying to match the functionality of how
Apple deals with grouped sections of text, since it seems to work
really well. Once we've prototyped that with xul, we can start
tweaking the behavior after playing around with it.
Here is a basic spec of the behavior I would like us to prototype:
Left and right arrows move the cursor between tags.
Hitting the backspace key selects the tag to the left of the
cursor, hitting it again deletes it.
Single clicking a tag selects it.
Double clicking a tag converts it back to selected text.
If a tag is in the text state and you create a new tag elsewhere, it
converts the text state tag back into the blue tag state.
Typing a new tag produces a drop down list of completions, the first
suggestion is filled out and selected.
Typing a comma or hitting enter creates the tag.
Hitting backspace when typing a new tag clears the selected area (as
expected) and also hides the current suggestions.
This field also has drag and drop functionality, but that isn't going
to be important for tagging since we are dealing with a single field.
Updated•18 years ago
|
Assignee: mano → swon
Comment 3•18 years ago
|
||
Here is a mockup of the ideal tag editor behavior, based on iteration 7 of the tagging and bookmarking UI.
See Dietrich's comment above for the text in the mockup.
Updated•18 years ago
|
Status: NEW → ASSIGNED
Comment 5•17 years ago
|
||
Another checkpoint.
Even though not perfect, most of the basic functionalities should be there other than autocomplete.
Attachment #273015 -
Attachment is obsolete: true
Reporter | ||
Comment 6•17 years ago
|
||
Autocomplete can be done in a follow up IMO.
Updated•17 years ago
|
Flags: blocking-firefox3? → blocking-firefox3-
Whiteboard: [wanted-firefox3]
Comment 7•17 years ago
|
||
This is still not perfect, but I think it covers most of the basic functionalities.
Mano, can I get some feedback on what's done so far and what needs to be added/fixed?
Attachment #273200 -
Attachment is obsolete: true
Updated•17 years ago
|
Attachment #274546 -
Flags: review?(mano)
Updated•17 years ago
|
Whiteboard: [wanted-firefox3] → [wanted-firefox3], [places-ui]
Updated•17 years ago
|
Assignee: stevewon → nobody
Status: ASSIGNED → NEW
Comment 8•17 years ago
|
||
Hey Mano.
Any update on the review?
Thanks.
Updated•17 years ago
|
Assignee: nobody → stevewon
Updated•17 years ago
|
Status: NEW → ASSIGNED
Comment 9•17 years ago
|
||
I tried the patch with patches from bug#385609 & bug#387486.
When integrated into the bookmarks property dialog, the problem with editor extending too far is seems partially fixed. Height seems a little too short. I will attach a screenshot.
When integrated into the starring popup, the events don't seem to get caught correctly. I will attach the patch for that.
Comment 10•17 years ago
|
||
Comment 11•17 years ago
|
||
Comment 12•17 years ago
|
||
Screenshot of Starring Popup after replacing tags textbox with editor.
Comment 13•17 years ago
|
||
RE: Comment #12
The bottom input box is supposed to be the editor.
So, there's still a problem.
Comment 14•17 years ago
|
||
When testing this make sure to test out larger fonts, since we may eventually try to use the same type of tag editor field in a KUI:
http://people.mozilla.com/~faaborg/files/20070705-kui/i1kuiTagging.png_large.png
Updated•17 years ago
|
Target Milestone: Firefox 3 M8 → Firefox 3 M9
Updated•17 years ago
|
Assignee: stevewon → nobody
Status: ASSIGNED → NEW
Updated•17 years ago
|
Target Milestone: Firefox 3 M9 → Firefox 3 M10
Updated•17 years ago
|
Target Milestone: Firefox 3 M10 → Firefox 3 M11
Updated•17 years ago
|
Version: unspecified → Trunk
Reporter | ||
Comment 15•17 years ago
|
||
Comment on attachment 274546 [details] [diff] [review]
Patch
Clearing review request until we can actually use the editor element in panels.
Attachment #274546 -
Flags: review?(mano)
Comment 16•17 years ago
|
||
Are you still expecting to land the tag editor within the Firefox 3 timeframe? If so, before this lands, I'd like to put it through manual a11y testing to make sure we don't introduce a lot of SEC508 issues:
1. Is the focus properly transitioning from the editor to the popup list and back?
2. is the popup list navigable using Up and DownArrows, and are proper a11y events fired so the new selection can be recognized by ATs?
etc.
Updated•17 years ago
|
Flags: wanted-firefox3+
Whiteboard: [wanted-firefox3], [places-ui] → , [places-ui]
Updated•17 years ago
|
Whiteboard: , [places-ui] → [places-ui]
Updated•17 years ago
|
Status: NEW → ASSIGNED
Updated•17 years ago
|
Target Milestone: Firefox 3 beta3 → ---
Updated•16 years ago
|
Assignee: tjduavis.opensource → nobody
Status: ASSIGNED → NEW
Priority: -- → P2
Comment 18•16 years ago
|
||
Anything which can be done here for Firefox 3.1?
Flags: blocking-firefox3.1?
Comment 19•16 years ago
|
||
Currently I'm reorganizing all my bookmarks and tagging them. As what I can see it's hard to do this without the auto-complete feature. It would be great to have this feature in the next version of Firefox. It's already supported by a lot of other applications and we shouldn't miss that.
Shall we create a new bug for only this feature if the work on the tag editor will takes too long?
Comment 20•16 years ago
|
||
That's bug 415960, I'd think. In the meantime use the "tweez" extension!
Updated•16 years ago
|
Flags: wanted-firefox3.1+
Flags: blocking-firefox3.1?
Flags: blocking-firefox3.1-
Updated•16 years ago
|
Target Milestone: --- → Firefox 3.2a1
Updated•16 years ago
|
Flags: wanted-firefox3.2?
Flags: blocking-firefox3.2?
Updated•16 years ago
|
Target Milestone: Firefox 3.6a1 → ---
Updated•16 years ago
|
Flags: wanted-firefox3.6?
Flags: wanted-firefox3.6+
Flags: wanted-firefox3.5+
Comment 22•15 years ago
|
||
Dietrich; moved the nomination to Firefox 3.7, dunno if you want to mark this in some way that indicates that it's some nice visual polish that we'd like to see done.
blocking2.0: --- → ?
Flags: blocking-firefox3.6? → blocking-firefox3.6-
Comment 23•15 years ago
|
||
Bug 451915 - move Firefox/Places bugs to Firefox/Bookmarks and History. Remove all bugspam from this move by filtering for the string "places-to-b-and-h".
In Thunderbird 3.0b, you do that as follows:
Tools | Message Filters
Make sure the correct account is selected. Click "New"
Conditions: Body contains places-to-b-and-h
Change the action to "Delete Message".
Select "Manually Run" from the dropdown at the top.
Click OK.
Select the filter in the list, make sure "Inbox" is selected at the bottom, and click "Run Now". This should delete all the bugspam. You can then delete the filter.
Gerv
Component: Places → Bookmarks & History
QA Contact: places → bookmarks
Comment 24•15 years ago
|
||
This is not a blocker for 1.9.3. I'd like to see this prototyped by an extension before determining if it's something we want in core. Definitely not blocking.
blocking2.0: ? → -
Comment 25•13 years ago
|
||
Removing uiwanted, faaborg's mockup seems to be a good direction — I'm unable to test the patch myself, but feel free to tag ux-review@mozilla.com for ui-review of this, if there's a tryserver build (or even a screencast) showing how it works.
Keywords: uiwanted
Updated•9 years ago
|
Priority: P2 → --
Comment 26•6 years ago
|
||
Per policy at https://wiki.mozilla.org/Bug_Triage/Projects/Bug_Handling/Bug_Husbandry#Inactive_Bugs. If this bug is not an enhancement request or a bug not present in a supported release of Firefox, then it may be reopened.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → INACTIVE
You need to log in
before you can comment on or make changes to this bug.
Description
•