Skip to content
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

Visual fixes on the 'Create new lesson' page #13133

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

MisRob
Copy link
Member

@MisRob MisRob commented Feb 28, 2025

Summary

Fixes width and spacing of inputs on the Create new lesson page. See commit messages.

Before After
Screenshot from 2025-02-28 18-22-51 Screenshot from 2025-02-28 18-22-39
Screenshot from 2025-02-28 18-23-12 Screenshot from 2025-02-28 18-23-21

References

Sub-task of #13127

Reviewer guidance

  • Go to "Create new lesson" page in Coach
  • Observe on various screen sizes

around the description input.

This was caused by -1em margin left
being applied to all textboxes instead
of only the title box.

This fix ensures that this style is
applied to the title box only, and only
on large screens. On smaller screens it
would cause issues with lack of space
between the input and icon.
@github-actions github-actions bot added APP: Coach Re: Coach App (lessons, quizzes, groups, reports, etc.) DEV: frontend labels Feb 28, 2025
@MisRob MisRob added the TODO: needs review Waiting for review label Feb 28, 2025
@MisRob MisRob requested review from radinamatic and pcenov February 28, 2025 17:46
@MisRob
Copy link
Member Author

MisRob commented Feb 28, 2025

cc @jtamiace do the screenshots after look okay? In the bugbash, we weren't 100% sure about the description field taking the whole width, however it seemed to be the only behavior that looked reasonable enough we could think of.

@jtamiace
Copy link
Member

It looks better; this is what I'm seeing on the demo server right now. Not sure if something happened along the way for 0.18 but I think returning it to this state would work.

Screenshot 2025-02-28 at 3 00 53 PM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
APP: Coach Re: Coach App (lessons, quizzes, groups, reports, etc.) DEV: frontend TODO: needs review Waiting for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants