Skip to content

Conversation

@mnocon
Copy link
Contributor

@mnocon mnocon commented Dec 15, 2025

@github-actions
Copy link

Preview of modified files

Preview of modified Markdown:

@mnocon mnocon force-pushed the add-crosslink-onlineeditor branch from 2671cbf to eff0e0d Compare December 15, 2025 19:36
@mnocon mnocon marked this pull request as ready for review December 16, 2025 11:43
@mnocon mnocon requested review from micszo and reithor December 16, 2025 11:43
@mnocon mnocon requested a review from a team December 16, 2025 12:11
@ibexa-workflow-automation-1 ibexa-workflow-automation-1 bot requested review from adriendupuis, dabrt and julitafalcondusza and removed request for a team December 16, 2025 12:11
When you select a section of text, you get access to text formatting options such as bold or underline.
Here you can also add a link to the text. You can link to an external website, or to another content item.

When linking to content, you can select a SiteAccess from the drop-down to enable cross-website linking.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The term SiteAccess can be hard to grasp for end-users. You could repeat the link used above to a definition of SA:
translate_content.md#siteaccess

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could a screen shot of the link "editor" with the site select unfold worth it?

Copy link

@reithor reithor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agree with :
@dabrt that link to SiteAccess explanation could be added.
@adriendupuis that a screenshot would be very beneficial

@reithor reithor self-requested a review December 17, 2025 10:51
@sonarqubecloud
Copy link

@mnocon
Copy link
Contributor Author

mnocon commented Dec 17, 2025

Thank you! New link and screenshot added in 87dea6d

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mnocon mnocon merged commit e7fde88 into 4.6 Dec 17, 2025
6 of 7 checks passed
@mnocon mnocon deleted the add-crosslink-onlineeditor branch December 17, 2025 14:06
mnocon added a commit that referenced this pull request Dec 17, 2025
* Added mention about cross-site linking

* Review feedback
@mnocon mnocon mentioned this pull request Feb 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants