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

Fix content tags modal #2641

Merged
merged 6 commits into from
Sep 1, 2021
Merged

Fix content tags modal #2641

merged 6 commits into from
Sep 1, 2021

Conversation

nzambello
Copy link
Member

Fixes: #2615

Adds:

  • array and token as named widgets in the config
    (this because I had conflicts in the widget retrieval due to choices attribute)

Fixes:

  • fixes correct behavior of the modal to change tags

Now you can correctly set or remove tags to one or more contents.
Retrieves Subject vocabulary to show options to set while you can also create a new one.
Shows contents' actual tags to remove.

@nzambello
Copy link
Member Author

Update: I added a fix in ContentsItem to show also array type indexes (eg: Subject).
Now tags are shown as comma separated values in theirs column (if that column is set, not default).

@sneridagh sneridagh merged commit 7ee0dcb into master Sep 1, 2021
@sneridagh sneridagh deleted the fix_content_tags_modal branch September 1, 2021 11:07
sneridagh added a commit that referenced this pull request Sep 1, 2021
* master:
  Back to development
  Release 13.13.0
  Prepare for release
  Showing version in History view (#2634)
  Fix SearchWidget required pathname (#2646)
  Fix content tags modal (#2641)
  Cut/Copy blocks: fixed cut/copy unselected blocks. (#2638)
  Fix QueryWidget styling (#2636)
  Bump tar from 6.1.6 to 6.1.11 (#2648)
  Bump tar from 6.1.8 to 6.1.11 in /packages/generator-volto (#2647)
  Scroll to top only if the location pathname changes (#2635)
  Disabled all the other configuration options when user did not choose… (#2629)
  Update pt_BR translation.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Tags (subjects) management in contents view partially works
4 participants