Skip to content

Fix unexpected line breaks for localized glossary tooltips#15920

Merged
k8s-ci-robot merged 1 commit intokubernetes:masterfrom
ttauveron:fix-line-breaks-glossary-tooltip
Aug 19, 2019
Merged

Fix unexpected line breaks for localized glossary tooltips#15920
k8s-ci-robot merged 1 commit intokubernetes:masterfrom
ttauveron:fix-line-breaks-glossary-tooltip

Conversation

@ttauveron
Copy link
Copy Markdown
Contributor

As described in #15390, there is an issue with line breaks and glossary tooltips.

In English, the tooltip is displayed correctly :

image

However, in another language, say French, there is a line break issue :

image

This is kind of a workaround, but it seems that removing the line breaks in layouts/partials/docs/glossary-terms.html fixes the issue...

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Aug 18, 2019
@k8s-ci-robot k8s-ci-robot added size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. sig/docs Categorizes an issue or PR as relevant to SIG Docs. labels Aug 18, 2019
@ttauveron ttauveron force-pushed the fix-line-breaks-glossary-tooltip branch from c362319 to b7dd48d Compare August 18, 2019 18:02
@k8s-ci-robot k8s-ci-robot added the do-not-merge/invalid-commit-message Indicates that a PR should not merge because it has an invalid commit message. label Aug 18, 2019
@ttauveron ttauveron force-pushed the fix-line-breaks-glossary-tooltip branch from b7dd48d to d20666d Compare August 18, 2019 18:06
@k8s-ci-robot k8s-ci-robot removed the do-not-merge/invalid-commit-message Indicates that a PR should not merge because it has an invalid commit message. label Aug 18, 2019
@netlify
Copy link
Copy Markdown

netlify bot commented Aug 18, 2019

Deploy preview for kubernetes-io-master-staging ready!

Built with commit c362319

https://deploy-preview-15920--kubernetes-io-master-staging.netlify.com

@netlify
Copy link
Copy Markdown

netlify bot commented Aug 18, 2019

Deploy preview for kubernetes-io-master-staging ready!

Built with commit d20666d

https://deploy-preview-15920--kubernetes-io-master-staging.netlify.com

@sftim
Copy link
Copy Markdown
Contributor

sftim commented Aug 18, 2019

/lgtm
(I haven't looked into how this works at the Hugo level)

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Aug 18, 2019
@tengqm
Copy link
Copy Markdown
Contributor

tengqm commented Aug 19, 2019

/approve

@k8s-ci-robot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: tengqm

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Aug 19, 2019
@k8s-ci-robot k8s-ci-robot merged commit bfe83d2 into kubernetes:master Aug 19, 2019
@ttauveron ttauveron deleted the fix-line-breaks-glossary-tooltip branch August 19, 2019 08:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm "Looks good to me", indicates that a PR is ready to be merged. sig/docs Categorizes an issue or PR as relevant to SIG Docs. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants