Closed Bug 374524 Opened 13 years ago Closed 12 years ago

Provide some UI that allows users to annotate a URI with tags

Categories

(Firefox :: Bookmarks & History, defect)

defect
Not set

Tracking

()

VERIFIED FIXED
Firefox 3

People

(Reporter: dietrich, Assigned: stevewon)

References

()

Details

(Whiteboard: PLCS-002d)

Attachments

(5 files)

This is a tracking bug for item PLCS-002d in the Firefox 3 PRD.
Blocks: 374945
Flags: blocking-firefox3?
Not blocking on P2 items, but really-really-want! :)
Flags: blocking-firefox3? → blocking-firefox3-
Whiteboard: PLCS-002d [wanted-firefox3]
Hello. Here's the prototype I've been working on for taggin. There have been a few emails going back and forth, but thought it would be better to share on bugzilla.

Here's the link to the prototype if anyone wants to try it out. 

http://www.andrew.cmu.edu/user/swon/mozilla/prototypes/tag_standalone1_v3.xul

The prototype focuses mainly on the textinput<->button interaction. 
A couple of things...
1. Selected tag will be in different color from the rest of the tags
2. You can traverse through tags using left and right arrows.
3. You can also select a tag by clicking on it.
4. You can edit the tag by double-clicking on the tag or by selecting it and pressing return.
5. You can use comma, tab, or return to create a tag.
6. You can select a tag and press backspace button to remove the tag.
7. If you are at the beginning of the textinput box, pressing backspace will select the tag to the left of the cursor, and pressing backspace again will delete it.


If you get a chance to play with it, any feedback would be helpful.. thanks! =]
(Tested on branch, slap me.)
1. Using the cursor to move between the tags disables navigation in the tag typed in (maybe use alt/ctrl + something for navigation which is not already used - if the tag is only used in the bookmark dialog, the commands for tab navigation could be used).
2. Pressing the "Insert" button while a tag is focused gives:
[Exception... "Component returned failure code: 0x80004005 (NS_ERROR_FAILURE) [nsIController.doCommand}" nsresult: "0x80004005 (NS_ERROR_FAILURE)" location: "JS frame :: chrome://global/content/globalOverlay.js :: anonymous :: line 156" data: no]
3. Left and top border are not very well to see, maybe the background of the box containing the button should be different
4. Pressing "Delete" on a tag should delete it, "Insert" should edit it (return open a tab with a list of all bookmarks containing the tab).
5. Jumping tags while switching the edited tag should be avoided.
Hey. 
For #1 (if I understand your comment correctly), you can move around in the textbox (the tag being written currently). When you hit left or right at the edge of the textbox, you will navigate to another tag.
#2, there's not handler set for "Insert" button.
#4, If a tag is selected, pressing "Delete" will delete it. If you press "Return" on a selected tag, you can edit the tag.
#5, not exactly sure what you mean.

Thanks for the feedback.
Attached file Prototype xul
> 5. You can use comma, tab, or return to create a tag.
When I use tab to create a tag, a new editable text field is created for the next tag but the focus jumps to the address bar, and I have to manually click back into the open textbox.

> 6. You can select a tag and press backspace button to remove the tag.
I'm testing on a trunk build on Mac OS X and doing this sends the browser back in history...

I do like how if you click a little to the right of the last created tag, the text field pops up automatically, though :)

> 7. If you are at the beginning of the textinput box, pressing backspace will
> select the tag to the left of the cursor, and pressing backspace again will
> delete it.

I... don't know why the browser's reacting the way it is to backspace, but the same problem comes up that I mentioned earlier, with the browser interpreting backspace as a back button.
Hey. haha the back button<=>backspace thing happened to me too,but didn't get a chance yet to figure that out.
One way you can test it instead is save the xul file, and drag it to the browser so it's the first thing it opens... heh.

The address bar thing never happened to me though.
Depends on: 279703
Attached file Tagging Extension
* Extension was tested on the Alpha 5 release.

* Setting up the extension

    1. Install the xpi file.

    2. (After restarting) Right click on toolbar and then “customize”.

    3. Find the tagging icon (titled as “Tagging UI Prototype 1″) and drag it to the toolbar.
**Patch wip.

I tried to use ifdef with this patch, but didn't turn out too well last time I tried (maybe I was doing something wrong). 

Also, currently just putting a span around favicon icon to click on it and get the popup up, but should change that too.
Assignee: nobody → swon
Depends on: 385266
Added mockups to bug 329261 (tag selector) and bug 387485 (tag editor).
Status: NEW → ASSIGNED
Target Milestone: --- → Firefox 3
fixed by the checkin of bug 385266.
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
verified with - Mozilla/5.0 (Macintosh; U; Intel Mac OS X; en-US; rv:1.9a8pre) Gecko/2007091304 Minefield/3.0a8pre
Status: RESOLVED → VERIFIED
Flags: wanted-firefox3+
Whiteboard: PLCS-002d [wanted-firefox3] → PLCS-002d
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
You need to log in before you can comment on or make changes to this bug.