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

Shadows and a correction #132

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

Conversation

jmfergeau
Copy link

This adds the shadow class for bootstra 4.4.1. This adds the shadow effect to basically any element you want, like in the original bootstrap. Also works with shadow-sm and shadow-lg with the same effect as fallback for bootstrap.

The shadow class used on cards elements
These are card elements with the shadow class added.

Note that it only works right with blocks that have a filled color like cards, alert boxes, jombotron, etc.

This also corrects a missing ; that prevents scss compilation.

@kristopolous
Copy link
Owner

thanks ... my notifications are basically endless floods of noise on github so sorry if I don't respond sooner. I'll review the code and get back to you.

@kristopolous
Copy link
Owner

kristopolous commented Aug 9, 2023

Alright reviewed, I'm going to ask you to replace your constants with the variable system that is used in the codebase. Please refer to the variables scss flie for reference.

Other than that, it's fine with one potential nice hack. The master class way of pulling this off would be through css filters and not merely a shadow css directive.

I forget whether I attempted this or not 3 years ago but in practice, some shadow implementations wouldn't occlude the background text but instead change the color scheme.

This is ONLY doable if you choose to have full-height bottom shadow. Right now you've gone with half-height. If that is the intention then there's still something that needs to be done.

You're simulating MS-ASCII 220 here. In this case you'll need to block out the bottom pixels because you're "replacing" it.
msdos_festival_ASCII

There's various ways of doing this that require a bizarre level of wizardry. I believe the source has some.

You can punt on all of these and just add the variables and I'll do them.

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.

2 participants