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

Updating CSS for Pizza Tracking feature in Data Tracker #156

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

Conversation

ChristinaTremel
Copy link

Added css for pizza tracker for #17747.

This code was copied from the Gitbook documentation that Susanne created for the Multipage step indicator and the only adjustment made was we increased the width of the progress bar in line 193 from 25% to 32%. This was because we have 3 list elements in this progress bar and Susanne had 4 list elements.

Added pizza tracker for issue #17747. This is specifically for the Project Intake Request form.
@ChristinaTremel ChristinaTremel added CSS Modified CSS code Knack Any Knack related PR's labels Jun 14, 2024
@ChristinaTremel ChristinaTremel self-assigned this Jun 14, 2024
Copy link
Contributor

@susannegov susannegov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looked over it and it looks good.

height: 10px;
margin-top: -2em;
}

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The closing bracket on line 269 is on its own, I would either get rid of the blank line 268 or add a comment on line 269 thats indicating it closes the @media mobile styling

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Made that change here 9359d02! Thanks Chrispin.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CSS Modified CSS code Knack Any Knack related PR's
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants