Skip to content

Conversation

@TylerZeroMaster TylerZeroMaster force-pushed the core-921-alegbra-1-pdf branch from 9008278 to 215b81a Compare July 21, 2025 14:23
- Make exercise images smaller
- Limit max-width to 95% so that the images never overlap the numbering
Currently only practice sections
New icon borrowed from cardboard with theme recolor
Come to find this note does not have title. That is kind of weird. This should work, though.

Tip: the icon is added to the first child and that first child will align with the icon.
These width/height settings weren't doing anything until I made it possible to adjust them.

I don't know why they were there to begin with.
@TylerZeroMaster TylerZeroMaster marked this pull request as ready for review August 25, 2025 15:33
Copy link
Contributor

@RoyEJohnson RoyEJohnson left a comment

Choose a reason for hiding this comment

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

I don't know what everything does, but it looks like a generally consistent approach.

@TylerZeroMaster
Copy link
Contributor Author

I don't know what everything does, but it looks like a generally consistent approach.

Thank you for reviewing this. Some of it is not as consistent as I would like. There were quite a few one-offs in algebra-1 that I was not sure would be used going forward. In those cases, I stuck with plain css instead of trying to create new reusable components/shapes.

You might have inferred this already, but most of these changes were adding the ability to override certain properties. You can only override a property if it is one of enum('ValueSet:::OPTIONAL') or enum('ValueSet:::REQUIRED'). In cases where the property value is like (enum('ValueSet:::OPTIONAL'), 10px) that is making it so that the default value is 10px but it can still be set manually if needed.

@TylerZeroMaster TylerZeroMaster merged commit 4d72a1e into main Aug 25, 2025
2 checks passed
@TylerZeroMaster TylerZeroMaster deleted the core-921-alegbra-1-pdf branch August 25, 2025 19:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants