Move text on button press

[fstr0]

Modify padding on active button to move the text 1px down and 1px right to replicate behaviour in Win98.

This fixes issue #34.

[DiskJunky]

I had the exact same fix. Just to note for posterity on some of the idiosyncrasies of the fix:

• 2px was required due to Chrome not moving a 1px value. There appears to be a conflicting rule in css resolution there
• 0px for the bottom as negative values aren't valid in FireFox and Chrome for padding. Interestingly, full padding-top, right, etc. does actually work with a negative value. Issue is specifically with the condensed padding tag and negative values for both browsers.

[jdan]

Is there a way to do this without padding? I worry about someone adding their own padding (for larger buttons) and forgetting to add/subtract one on :active

[jdan]

Padding Q

[tpenguinltg]

Perhaps you can make use of an invisible border instead since box-sizing is set to border-box? As far as I can tell, the border has been removed on buttons anyway, and I don't see any legitimate use for a downstream user to add it back if they want to keep the theme.

[michaeldll]

Perhaps you can make use of an invisible border instead since box-sizing is set to border-box? As far as I can tell, the border has been removed on buttons anyway, and I don't see any legitimate use for a downstream user to add it back if they want to keep the theme.

I had a go at implementing your suggestion on #79 :)

[jdan]

@fstr0 I'm happy to merge this as-is, since I don't think we can do better than adjusting the padding :) Wanna rebase off master?

[vercel]

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/jdan/98css/1m4i1tq0b
✅ Preview: https://98css-git-fork-fstr0-master.jdan.now.sh

[fstr0]

@jdan I think that is done now? I'm a GitHub noob so let me know if I did it wrong

[jdan]

Hey @fstr0,

Sorry for the pain! We've removed build/98.css and docs/98.css from version control, so they'll need to be git rmd before I merge.

[jdan]

@fstr0 thanks so much <3