nextcloud-deck icon indicating copy to clipboard operation
nextcloud-deck copied to clipboard

Tag dropdown in card editor doesn't show tags which were removed in same edit activity.

Open ghost opened this issue 3 years ago • 1 comments

Please use GitHub reactions 👍 to show that you are affected by the same issue. Please don't comment if you have no relevant information to add!

Describe the bug

To reproduce Issue is reproducible with an account from try.nextcloud.com:

  • [X] Yes (Haven't tested, but this probably is unrelated to account information.)
  • [ ] No

Steps to reproduce the behavior:

  1. Open Deck app on smartphone.
  2. Click on any card that already has tags to open it for editing.
  3. Click on the tag dropdown for adding tags.
  4. It shows only tags that are not yet added to the card. (which is good)
  5. Remove one of the tags that is already present on the card.
  6. Click on the tag dropdown for adding tags again.
  7. The dropdown shows the identical collection of tags compared to the first time. The removed one is missing. (Bad)

Expected behavior

Tags which have been removed while editing the card should show up in the add-tag-dropdown right after they've been removed, so they can be readded in the same edit activity, without discarding or saving the cards once in between an reopening the card editor again.

Screenshots

-none-

Versions

  • Nextcloud: 20.0.6
  • Nextcloud Deck: 1.2.5
  • Nextcloud Android: 3.15.0
  • Nextcloud Android Deck: 1.14.5

Smartphone (please complete the following information):

  • Device: Fairphone 2
  • Android-Version: LineageOS 17.1 (Android 10)
  • App-Store:
    • [x] Google Play Store
    • [ ] Google Play Store (Beta channel)
    • [x] F-Droid
    • [ ] Huawei AppGallery

Are you using LDAP?

  • [ ] Yes
  • [x] No

Stacktrace -none-

ghost avatar Mar 07 '21 10:03 ghost

Hi @mrbenjoi, wow, very good catch! And very nice detailed explanation, we'll have a look at it!

@stefan-niedermann I haven't any chance to include the deleted one, because it is deleted at save. I guess you'll need to handle this on the UI...

desperateCoder avatar Mar 07 '21 10:03 desperateCoder