hacker icon indicating copy to clipboard operation
hacker copied to clipboard

Add styles for tasklist

Open davorpa opened this issue 4 years ago • 11 comments

Generate styles for tasklist preserving theme colors

image to image

A complete implement for pages-themes/hacker#75. Resolves #75

Preview a demo at https://davorpa.github.io/github-pages-themes-hacker/#what-about-task-lists

davorpa avatar Aug 31 '21 03:08 davorpa

@parkr could you approval me to workflow?

It works like a charm and my build pass. See davorpa/github-pages-themes-hacker#5

davorpa avatar Aug 31 '21 03:08 davorpa

This is a great idea and simple implementation.

jimhall avatar Aug 31 '21 18:08 jimhall

This LGTM, though it would be great to test this on all major browsers.

  • What is the pointcut?

Some feedback in my scope.

Windows - Chrome

As seen as in PR description. I dont remember the version I took the snapshot with :worried:. Now I have 94.0.4606.81 and it looks the same.

Windows - Edge 94.0.992.38

image

Windows - IE 11.0.9600.19596 or minor

image

Need fix. Maybe due to missing ms- vendor property Not supported :cry:

  • https://css-tricks.com/almanac/properties/a/appearance/
  • https://caniuse.com/mdn-css_properties_appearance_none

Here some pointcut CSS hacks: https://browserstrangeness.bitbucket.io/css_hacks.html

davorpa avatar Oct 13 '21 02:10 davorpa

Still better than what we started with!

jimhall avatar Oct 13 '21 12:10 jimhall

I believe Microsoft is trying to migrate to Edge and away from Internet Explorer, so this shouldn't be a huge issue.

Safari, Chrome, and Firefox on Mac look good to me.

:shipit:

parkr avatar Oct 13 '21 17:10 parkr

(I left GitHub and can't merge this, but I give it my approval!)

parkr avatar Oct 13 '21 17:10 parkr

Still better than what we started with!

I have in my backlog apply some IE11- hack to remove this decoration. I'll also try to increase checkboxes size to be pixel perfect more or less.

davorpa avatar Oct 13 '21 19:10 davorpa

در تاریخ چهارشنبه ۱۳ اکتبر ۲۰۲۱،‏ ۲۳:۱۰ David Ordás < @.***> نوشت:

Still better than what we started with!

I have in my backlog apply some IE11- hack to remove this decoration. I'll also try to increase checkboxes size to be pixel perfect more or less.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/pages-themes/hacker/pull/83#issuecomment-942657218, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOOITSZLOC6I3O7AUW5JWFLUGXOELANCNFSM5DDFHWQQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

rabooli avatar Oct 13 '21 20:10 rabooli

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Jan 09 '22 01:01 stale[bot]

Would prefer this kept open.

jimhall avatar Jan 09 '22 01:01 jimhall

Bump

davorpa avatar Jan 09 '22 18:01 davorpa