IBX-10684: Document translation management - #407
Conversation
Preview of modified filesPreview of modified Markdown: |
Summary
Errors per inputErrors in site/content_management/translate_content/index.html
|
juskora
left a comment
There was a problem hiding this comment.
I think it misses mention a Workflow status in the Versions tab following:
https://github.com/ibexa/workflow/pull/190
9796c43 to
f38223e
Compare
There was a problem hiding this comment.
This screenshot also needs an update, the drop-down is different.
There was a problem hiding this comment.
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.
@julitafalcondusza WDYT?
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
The cursor does not make it to the screenshot. The image in comment is just to show you how the pulldown behaves
There was a problem hiding this comment.
Got it.
I'd keep the one with search field and add some info about it (appears when at least 4 services configured).
There was a problem hiding this comment.
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
left a comment
There was a problem hiding this comment.
Please mention policies that allow editor user use AI providers also and add info that translations can be in system workflow.
julitafalcondusza
left a comment
There was a problem hiding this comment.
Note that this part or review only "touches" the screenshots.
There was a problem hiding this comment.
Got it.
I'd keep the one with search field and add some info about it (appears when at least 4 services configured).
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
Quality is poor (frame looks much wider)
plus: what about adding step-circles from Snagit here?
There was a problem hiding this comment.
Why is this view so condensed? The text wraps as if the window had been resized. A wider view would look much better.
There was a problem hiding this comment.
This looks really bad: the image quality and cropping are off.
Please show wider context, add steps (available in Snagit) and frame.
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