-
Notifications
You must be signed in to change notification settings - Fork 290
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
Fixes #37839 - bold skip dependency solving checkbox text #11151
Conversation
b40ba5a
to
bd7ef94
Compare
@@ -119,7 +119,7 @@ <h3 ng-show="errata" translate>Apply {{ errata.errata_id }}</h3> | |||
<div class="checkbox" ng-show="updates.length > 0"> | |||
<label> | |||
<input name="noDepSolve" ng-model="noDepSolve" type="checkbox"/> | |||
<span translate>Skip dependency solving for a significant speed increase. If the update cannot be applied to the host, delete the incremental content view version and retry the application with dependency solving turned on.</span> | |||
<span translate><b>Skip dependency solving for a significant speed increase</b>. If the update cannot be applied to the host, delete the incremental content view version and retry the application with dependency solving turned on.</span> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not a 100% sure if translation would work here..Maybe something like this?
....<span ...../>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see two options to fix this. Either I split up the string into a <b translate>
block and a <span translate>
block, or I just bold the entire thing.
Bolding the entire thing would change the look of course. Splitting it into two blocks would make the strings re-require retranslation.
Unless there's a potential fix that I'm not seeing where we can selectively bold and keep the translation string together?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bd7ef94
to
4a13690
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. Brings attention to the text without messing up current translations.. 👍🏼
What are the changes introduced in this pull request?
Bolds "Skip dependency solving for a significant speed increase" text when performing an incremental update via the UI.
Considerations taken when implementing this change?
Should all or some of the text be bold?
What are the testing steps for this pull request?
Try to apply some errata to a host that is applicable but not installable. Decide which looks better:
or