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 link in pop-up RelationsMatrix.jsx #6085

Merged
merged 2 commits into from
Jun 11, 2024

Conversation

stevepiercy
Copy link
Collaborator

@stevepiercy stevepiercy commented Jun 9, 2024

This link is outdated after #5809, and its anchor never existed. This PR corrects both issues.


📚 Documentation preview 📚: https://volto--6085.org.readthedocs.build/

This link is outdated after #5809, and its anchor never existed. This PR corrects both issues.
Copy link

netlify bot commented Jun 9, 2024

Deploy Preview for plone-components canceled.

Name Link
🔨 Latest commit ee4962a
🔍 Latest deploy log https://app.netlify.com/sites/plone-components/deploys/66656107d415a700084e559d

@stevepiercy stevepiercy self-assigned this Jun 9, 2024
@ichim-david
Copy link
Member

@stevepiercy the selection of the toc is a bit strange with your link
relation-fields
It shows in the toc the previous section selected when you open the link.
I suggest we simply use: https://6.docs.plone.org/volto/development/widget.html#relation-fields
then it shows up like this:
relation-fields-toc

@stevepiercy
Copy link
Collaborator Author

@ichim-david I get different results. The id is for the span above the h2. I don't know how you get different results.

Screenshot 2024-06-09 at 7 17 56 AM

@stevepiercy
Copy link
Collaborator Author

Also in Sphinx and MyST, it is preferred to use the label, not the heading text, as that heading text may change, but the label almost never does. When the label changes, it is much easier to do a find and replace.

@stevepiercy
Copy link
Collaborator Author

@ksuess is the link with its anchor correct? I made an educated guess, but I'd like your feedback.

@ksuess
Copy link
Member

ksuess commented Jun 11, 2024

Yes, https://6.docs.plone.org/volto/development/widget.html#widget-relation-field-label where the info is located. Thank you for bringing this up and taking time to find a solution!

@stevepiercy stevepiercy merged commit c08e7a3 into main Jun 11, 2024
70 checks passed
@stevepiercy stevepiercy deleted the fix-link-in-relationsMatrix-popup branch June 11, 2024 22:25
sneridagh added a commit that referenced this pull request Jun 12, 2024
* main:
  Fix link in pop-up RelationsMatrix.jsx (#6085)
  Fix deselecting lists. (#6080)
sneridagh added a commit that referenced this pull request Jun 15, 2024
* main: (63 commits)
  Release 18.0.0-alpha.35
  Release @plone/types 1.0.0-alpha.16
  (fix): export getFieldURL from Url.js in helpers (#6100)
  Improve container detection, `config.settings.containerBlockTypes` is no longer needed (#6099)
  Support nested directories in public folder add-on sync folders both … (#6098)
  Release 18.0.0-alpha.34
  Add generated new declarations for a34, just in case (#6097)
  Release @plone/slate 18.0.0-alpha.13
  Release @plone/registry 1.6.0
  Release @plone/types 1.0.0-alpha.15
  Add support for reading the add-ons `tsconfig.json` paths (#6096)
  Fixes #6046 pass proper defaults to align and size fields of Image block (#6093)
  bug fix. relations control panel. Restrict eglible relation targets a… (#6092)
  Fix Uncaught RangeError: date value is not finite in DateTimeFormat.format (#6088)
  Add optional `token` parameter to ploneClient initialization (#6077)
  Blocks Layout Navigator (#5642)
  Fix internalUrl  Widget to Reflect Prop Changes via onChangeBlock (#6036)
  [types] Improve styleClassNameExtenders types, Icon component JSDoc t… (#6095)
  Fix link in pop-up RelationsMatrix.jsx (#6085)
  Fix deselecting lists. (#6080)
  ...
sneridagh added a commit to aryan7081/volto that referenced this pull request Jun 15, 2024
* main: (42 commits)
  Release 18.0.0-alpha.35
  Release @plone/types 1.0.0-alpha.16
  (fix): export getFieldURL from Url.js in helpers (plone#6100)
  Improve container detection, `config.settings.containerBlockTypes` is no longer needed (plone#6099)
  Support nested directories in public folder add-on sync folders both … (plone#6098)
  Release 18.0.0-alpha.34
  Add generated new declarations for a34, just in case (plone#6097)
  Release @plone/slate 18.0.0-alpha.13
  Release @plone/registry 1.6.0
  Release @plone/types 1.0.0-alpha.15
  Add support for reading the add-ons `tsconfig.json` paths (plone#6096)
  Fixes plone#6046 pass proper defaults to align and size fields of Image block (plone#6093)
  bug fix. relations control panel. Restrict eglible relation targets a… (plone#6092)
  Fix Uncaught RangeError: date value is not finite in DateTimeFormat.format (plone#6088)
  Add optional `token` parameter to ploneClient initialization (plone#6077)
  Blocks Layout Navigator (plone#5642)
  Fix internalUrl  Widget to Reflect Prop Changes via onChangeBlock (plone#6036)
  [types] Improve styleClassNameExtenders types, Icon component JSDoc t… (plone#6095)
  Fix link in pop-up RelationsMatrix.jsx (plone#6085)
  Fix deselecting lists. (plone#6080)
  ...
sneridagh added a commit that referenced this pull request Jun 17, 2024
* main: (31 commits)
  Release 18.0.0-alpha.35
  Release @plone/types 1.0.0-alpha.16
  (fix): export getFieldURL from Url.js in helpers (#6100)
  Improve container detection, `config.settings.containerBlockTypes` is no longer needed (#6099)
  Support nested directories in public folder add-on sync folders both … (#6098)
  Release 18.0.0-alpha.34
  Add generated new declarations for a34, just in case (#6097)
  Release @plone/slate 18.0.0-alpha.13
  Release @plone/registry 1.6.0
  Release @plone/types 1.0.0-alpha.15
  Add support for reading the add-ons `tsconfig.json` paths (#6096)
  Fixes #6046 pass proper defaults to align and size fields of Image block (#6093)
  bug fix. relations control panel. Restrict eglible relation targets a… (#6092)
  Fix Uncaught RangeError: date value is not finite in DateTimeFormat.format (#6088)
  Add optional `token` parameter to ploneClient initialization (#6077)
  Blocks Layout Navigator (#5642)
  Fix internalUrl  Widget to Reflect Prop Changes via onChangeBlock (#6036)
  [types] Improve styleClassNameExtenders types, Icon component JSDoc t… (#6095)
  Fix link in pop-up RelationsMatrix.jsx (#6085)
  Fix deselecting lists. (#6080)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants