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

Renaming a Tag adds it to SelectedNoteTags list regardless if such a NoteTag exists #2618

Closed
vsimkus opened this issue Feb 29, 2020 · 4 comments
Labels
bug It's a bug desktop All desktop platforms high High priority issues

Comments

@vsimkus
Copy link
Contributor

vsimkus commented Feb 29, 2020

Environment

Joplin version: 1.0.178
Platform: Linux
OS specifics: Debian Bullseye (Testing)

Steps to reproduce

  1. Create a note and attach a tag.
  2. Create another note without tags.
  3. Rename the tag using the sidebar context menu.

Renaming a tag adds it to the top editor bar even though the note is not tagged with this particular tag. The note is not actually tagged, it only shows up in the top bar. If you switch the selected note to another one and back, then the tag will have disappeared from the top bar.

Describe what you expected to happen

Renaming a tag that is not associated with the currently selected note should not add the tag item to the top bar in the editor.

Screencast

The screencast doesn't show the renaming dialog, but I just change the name of the tag from "coding" to "coding_2".
demo

@vsimkus vsimkus added the bug It's a bug label Feb 29, 2020
@laurent22 laurent22 added high High priority issues desktop All desktop platforms labels Feb 29, 2020
@laurent22
Copy link
Owner

Indeed there's a bug here.

@Rishabh-malhotraa
Copy link
Contributor

@vsimkus I'll look into it, thanks for pointing it out

@PRASHANT1104
Copy link

@Rishgod are you working on it ? If not , I would like to work on it.

@Chhekur
Copy link

Chhekur commented Mar 7, 2020

@laurent22 I have fixed this issue and made a PR : #2680

Chhekur added a commit to Chhekur/joplin that referenced this issue Mar 8, 2020
@lock lock bot locked and limited conversation to collaborators Mar 22, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug It's a bug desktop All desktop platforms high High priority issues
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants