-
Notifications
You must be signed in to change notification settings - Fork 479
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
Show tutorial section for english only on code.org/music #58700
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.
One question to make sure I'm understanding correctly, but otherwise lgtm!
%section | ||
.wrapper | ||
.flex-container.justify-space-between.align-items-center.wrap.gap-2 | ||
%figure.col-47 |
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 take it this part changed because you're making space for the additional 'Available in English only' line?
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.
Correct! Adding that line made the text part taller than the image so adjusting the widths of the image and text wrapper makes the section look more balanced and easier to read.
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.
My Kelby-senses are improving by the day
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.
😁 🙌
Shows the Tutorial section for English-only on https://code.org/music.
Links
Jira ticket: ACQ-1894
Slack convo: here
Testing story
Tested locally in English and French
Followup
We will show this section to all languages once the "Available in English only" string translation comes in. See followup ticket: ACQ-1895
English
French