Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support tag renaming #1396

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

yuzawa-san
Copy link
Contributor

based on the comment "The problem is that after the tag name is changed in GUI you can not find the right TagInfo because you dont know the old tag name." i found that if we used a dialog like we did in #1395 rather than the row/cell edit mechanism, then we would not have that problem and could do something similar to what was being done in BookmarksTagList::changeColor. i also needed to emit the TagListChanged from the bookmarks since i found that editing the color in all 3 places (tags list, new bookmark dialog, bookmarks tags dialog) would not consistently update the color in the other places.

this likely supersedes #1165

@vladisslav2011
Copy link
Contributor

Hello.
Issues:

  1. It looks like it is possible to rename "Untagged" tag.
  2. It is possible to rename a tag to another tag's name.

It may be better to fix inline renaming instead of adding a new popup.
I've tried it and found that it is not really hard. See these commits:
vladisslav2011@684b145
vladisslav2011@34897c8
vladisslav2011@73142fa
vladisslav2011@20a7568
vladisslav2011@38bfeb6
vladisslav2011@c33fb0a

@vladisslav2011
Copy link
Contributor

It is still possible to edit the tag inline with inconsistent results...

My attempt to implement tag renaming is here #1398
I think, it still has some minor issues (it may be possible to create multiple "new" tags, that will collapse when assigned to a bookmark for example), so it is possible, that I'll update it soon.

@yuzawa-san
Copy link
Contributor Author

could you please elaborate? is the solution to drop the Qt::ItemIsEditable? all i see if that there is still some oddness maybe around creating a new tag and not attaching it to any bookmarks, but that predates this pr. the solution would be to write the tags to the config regardless of their attachment to any bookmark.

the *new* tag behavior should be fixed in #1395

@vladisslav2011
Copy link
Contributor

is the solution to drop the Qt::ItemIsEditable?

I have not tested it, but I think, it will prevent inline editing.

@vladisslav2011
Copy link
Contributor

Hmmm. It may be possible to get rid of modal dialogs in the bookmarks dock completely by adding a mode switcher (a combo box or a check box) to switch tag list mode between "Filter by tags" and "Edit selected bookmark's tags".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants