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
Bug 1881898: fix Quickstart subtitle block #6692
Bug 1881898: fix Quickstart subtitle block #6692
Conversation
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.
The bug also mentions that the duration shown after the quick start title should not wrap around to next line. In your screenshots I only see the issue resolved for task titles.
3cad0a7
to
0f55ad0
Compare
@rohitkrai03 I've added the Header part changes and also updated the screenshots. However, it looks a little odd in some cases. |
@abhinandan13jan: This pull request references Bugzilla bug 1881898, which is valid. The bug has been moved to the POST state. 3 validation(s) were run on this bug
In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/kind bug |
@abhinandan13jan Yeah, looking at the screenshots it seems the text is wrapping at wrong places. Not sure if using |
0f55ad0
to
02c71ec
Compare
@abhinandan13jan: This pull request references Bugzilla bug 1881898, which is invalid:
Comment In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
<span className="co-quick-start-drawer__duration"> | ||
<Text component={TextVariants.small} className="text-secondary"> | ||
{`${quickStart?.spec.duration} minutes`} | ||
</Text> | ||
</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.
Why are you using Text
? Fairly certain Text
is only useful inside of TextContent
.
<span className="co-quick-start-drawer__duration"> | |
<Text component={TextVariants.small} className="text-secondary"> | |
{`${quickStart?.spec.duration} minutes`} | |
</Text> | |
</span> | |
<small className="co-quick-start-drawer__duration text-secondary"> | |
{`${quickStart?.spec.duration} minutes`} | |
</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.
updated
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 was using Text as I simply moved a pre-existing block inside the Header without modifying it
<Text component={TextVariants.a} className="text-secondary"> | ||
{' '} | ||
{subtitle} | ||
</Text> |
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.
You created a class name never used it. Also you need to remove the space from the inline-block element.
<Text component={TextVariants.a} className="text-secondary"> | |
{' '} | |
{subtitle} | |
</Text> | |
<> | |
{' '} | |
<small className="co-quick-start-task-header__subtitle text-secondary"> | |
{subtitle} | |
</small> | |
</> |
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.
updated
02c71ec
to
ce22aaa
Compare
/bugzilla refresh |
@abhinandan13jan: This pull request references Bugzilla bug 1881898, which is valid. 3 validation(s) were run on this bug
In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/approve |
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.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: abhinandan13jan, christianvogt, rohitkrai03 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/retest |
/retest Please review the full test history for this PR and help us cut down flakes. |
10 similar comments
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
@abhinandan13jan: All pull requests linked via external trackers have merged: Bugzilla bug 1881898 has been moved to the MODIFIED state. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Addresses
https://issues.redhat.com/browse/ODC-4779
Problem
The sub title of the main and step headers are misaligned when the header text is wrapping to another line
Solution
Screenshots
Tests
Altered tests QuickStartHeader.spec.scss
Browser conformance
Chrome, Firefox