Skip to content

IBX-10684: Document translation management - #407

Open
dabrt wants to merge 20 commits into
5.0from
translations-management
Open

IBX-10684: Document translation management#407
dabrt wants to merge 20 commits into
5.0from
translations-management

Conversation

@dabrt

@dabrt dabrt commented Jun 9, 2026

Copy link
Copy Markdown
Contributor
Question Answer
JIRA Ticket IBX-10684
Versions 5.x
Edition

Document translation management in user doc.

Counterpart of ibexa/documentation-developer#3249

PR waits with images and specific procedures for UI changes to the product.

Checklist

  • Text renders correctly
  • Text has been checked with vale

@dabrt
dabrt requested a review from juskora June 9, 2026 14:55
@github-actions

github-actions Bot commented Jun 9, 2026

Copy link
Copy Markdown

Comment thread docs/content_management/translate_content.md Outdated
@dabrt
dabrt requested a review from juskora June 11, 2026 12:26
@dabrt
dabrt marked this pull request as draft June 19, 2026 07:45
@dabrt
dabrt marked this pull request as ready for review June 19, 2026 08:30
@dabrt
dabrt requested review from AnnaKaih and mikadamczyk June 19, 2026 08:30
@dabrt
dabrt requested a review from ciastektk June 19, 2026 08:51
Comment thread docs/content_management/translate_content.md Outdated
Comment thread docs/content_management/translate_content.md Outdated
Comment thread docs/content_management/translate_content.md Outdated
@dabrt
dabrt requested a review from mikadamczyk July 21, 2026 07:40
Comment thread docs/content_management/translate_content.md
@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown

Summary

Status Count
🔍 Total 13261
🔗 Unique 1320
✅ Successful 511
⏳ Timeouts 0
🔀 Redirected 0
👻 Excluded 12748
❓ Unknown 0
🚫 Errors 2
⛔ Unsupported 0

Errors per input

Errors in site/content_management/translate_content/index.html

  • [ERROR] file:///home/runner/work/documentation-user/documentation-user/repositories/devdoc-5.0/site/multisite/languages/translations_management/index.html (at 2437:52) | File not found. Check if file exists and path is correct
  • [ERROR] file:///home/runner/work/documentation-user/documentation-user/repositories/devdoc-5.0/site/multisite/translations_management/configure_translations_management/index.html#configure-translation-providers (at 2454:67) | File not found. Check if file exists and path is correct

Full Github Actions output

@dabrt
dabrt requested a review from ciastektk August 7, 2026 13:27
Comment thread docs/content_management/translate_content.md Outdated
Comment thread docs/content_management/translate_content.md Outdated
Comment thread docs/content_management/translate_content.md Outdated

@juskora juskora left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think it misses mention a Workflow status in the Versions tab following:
https://github.com/ibexa/workflow/pull/190

Comment thread docs/content_management/translate_content.md Outdated
Comment thread docs/content_management/translate_content.md Outdated
@dabrt
dabrt force-pushed the translations-management branch from 9796c43 to f38223e Compare August 12, 2026 15:28
Comment thread docs/content_management/img/create_translation.png Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This screenshot also needs an update, the drop-down is different.

@dabrt dabrt Aug 13, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

That modal displays a search field when there are more than four services. I'd rather leave the version without the search field to save people from wondering why they can't see it when they have only two providers configured in the system.

This way, when they have more providers, there's a bonus to be happy about.

obraz

@julitafalcondusza WDYT?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

First, a "technical" thing - we don't leave cursors in screenshots (if we do, it's exceptional and with a good reason).
I'll be able to comment on the context of the screenshot once I do a full review of the PR.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The cursor does not make it to the screenshot. The image in comment is just to show you how the pulldown behaves

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Got it.
I'd keep the one with search field and add some info about it (appears when at least 4 services configured).

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

plus: quality is really poor
It’s also better to avoid cropping the image in a way that leaves a “darker frame” from the background. It's better if only the white window is visible (even though the drop-down causes that some of the darker background is still visible at the bottom), or to change the crop and show a larger context of the image.

@juskora juskora left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please mention policies that allow editor user use AI providers also and add info that translations can be in system workflow.

Comment thread docs/content_management/translate_content.md Outdated

@julitafalcondusza julitafalcondusza left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Note that this part or review only "touches" the screenshots.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Got it.
I'd keep the one with search field and add some info about it (appears when at least 4 services configured).

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

plus: quality is really poor
It’s also better to avoid cropping the image in a way that leaves a “darker frame” from the background. It's better if only the white window is visible (even though the drop-down causes that some of the darker background is still visible at the bottom), or to change the crop and show a larger context of the image.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Quality is poor (frame looks much wider)
plus: what about adding step-circles from Snagit here?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why is this view so condensed? The text wraps as if the window had been resized. A wider view would look much better.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This looks really bad: the image quality and cropping are off.
Please show wider context, add steps (available in Snagit) and frame.

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.

6 participants