Swap box-shadows with actual borders

[robert-ryu7]

No description provided.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated
98css	✅ Ready (Inspect)	Visit Preview	Nov 16, 2022 at 8:12AM (UTC)

[jdan]

How come?

…

On November 16, 2022, Mads Hougesen ***@***.***> wrote: You can view, comment on, or merge this pull request online at:   <#154> Commit Summary * b957c65 <b957c65> Swap box-shadows with actual borders File Changes (7 files <https://github.com/jdan/98.css/pull/154/files>) * M docs/docs.css <https://github.com/jdan/98.css/pull/154/files#diff- 1c4b5ab645aeed797656ae69e9b7ce1a9852dd197e60e78bfe11b2b9e3aabefe> (4)  * A icon/border-button-raised.svg <https://github.com/jdan/98.css/pull/154/files#diff- 1ca6b2c96ec3d2e1ff9490cd10aa2628b22bad86cf4ae4869841a8514e4ba256> (10)  * A icon/border-button-sunken.svg <https://github.com/jdan/98.css/pull/154/files#diff- 5c68bf8e9ad3ac5d6c86836101d54f30a4251aadbad33c1fdb10ba2916e3b414> (10)  * A icon/border-raised.svg <https://github.com/jdan/98.css/pull/154/files#diff- 93eab077218918a361b46e875168bdb21e5ca8eb1fd238d3e7be7568ec5ba34a> (10)  * R icon/border-sunken.svg <https://github.com/jdan/98.css/pull/154/files#diff- aff81a47bc88af340c2e18923f80a9e6060c70732d7fb73f2dff4bb529552403> (0)  * A icon/border-table-header-sunken.svg <https://github.com/jdan/98.css/pull/154/files#diff- eef152e0640201a3f17484449727bd133b732cf09e8d2542de18e994f50abea5> (6)  * M style.css <https://github.com/jdan/98.css/pull/154/files#diff- b78be019f1dc6d57753ea900c3805b114cd53ab7c0db836cc081836df1b99b7a> (102)  Patch Links: * <https://github.com/jdan/98.css/pull/154.patch> * <https://github.com/jdan/98.css/pull/154.diff> — Reply to this email directly, view it on GitHub <#154>, or unsubscribe <https://github.com/notifications/unsubscribe- auth/AACGEJG7B7LPQ4PZGAMZUXTWISJOFANCNFSM6AAAAAASB5ZGGI>. You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>

[robert-ryu7]

Hi @jdan,

You mentioned previously that you prefer consistency (as do I). After giving it some thought I chose to use border image technique for all borders.

border-image uses actual CSS border which allows us to properly position scrollbars inside of an element (i did some comparisons using Win98 VM and scrollbars should be next to border and not over it). The only alternative solution that I know is to use 2 elements, one for overflow and other for border which might be ok but feels too cumbersome.

However, this makes theming a bit more difficult (but not impossible) since you won't be able to simply modify some color variables.

Anyway, if that's too much feel free to close this PR, don't want to go against any project rules. :)

[juanigaray]

I'm gonna close this PR as stale as a housekeeping measure. Please feel free to start a discussion in the discussions or issues sections.