BOOTSTRA.386 icon indicating copy to clipboard operation
BOOTSTRA.386 copied to clipboard

Shadows and a correction

Open jmfergeau opened this issue 2 years ago • 2 comments

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.

jmfergeau avatar Jul 31 '23 11:07 jmfergeau

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 avatar Aug 09 '23 16:08 kristopolous

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.

kristopolous avatar Aug 09 '23 16:08 kristopolous