Update pi-hole.css

[XhmikosR]

• format code
• use background-color instead of the background shorthand
• use the animation shorthand
• only transition opacity and transform in the chart tooltip
• add browserslist settings in package.json

By submitting this pull request, I confirm the following: {please fill any appropriate checkboxes, e.g: [X]}

{Please ensure that your pull request is for the 'devel' branch!}

• I have read and understood the contributors guide, as well as this entire template.
• I have made only one major change in my proposed changes.
• I have commented my proposed changes within the code.
• I have tested my proposed changes.
• I am willing to help maintain this change if there are issues with it later.
• I give this submission freely and claim no ownership.
• It is compatible with the EUPL 1.2 license
• I have squashed any insignificant commits. (git rebase)
• I have Signed Off all commits. (git commit --signoff)

---

This is untested

It will conflict with #1094 so I'll need to rebase this later. We might also want to wait until we land #1080 and #1079 so that we do this once for the CSS part.

Non-whitespace diff: https://github.com/pi-hole/AdminLTE/pull/1097/files?w=1

[XhmikosR]

I think I'm gonna redo this after we all agree on some sensible browserslistrc. There's no point in supporting IE < 9 or even 10, or Opera Mini since these should not work already anyway due to JS.

[XhmikosR]

@dschaper can we agree on a more realistic browserslistrc? Then I will update this and remove any obsolete vendor prefixes from our CSS.

[dschaper]

Sure, @PromoFaux which IE do we still need?

[PromoFaux]

IE10 and 11, though, 10 becomes EOL at the end of this month. I can't imagine there are too many enterprise environments out there running <11.

[dschaper]

I think a min of 11 is good.

[XhmikosR]

IMHO we should target IE >= 10. IE 9 might also work, but I cannot test it personally.

Plus the normal evergreen browsers.

See https://browsersl.ist/?q=defaults%2C+IE+%3E%3D+10%2C+not+ie_mob+%3C%3D+11 and play with it to visualize the browsers.

[dschaper]

If 10 is supportable then great. I was operating under the assumption that 11 would be uncomfortable to shim in but 10 would be like ripping off a band-aid.

If it's not to much more then 10 would be great.

[XhmikosR]

It should work, but I cannot confirm if the current devel branch works with IE 10. As long as we don't use new features, we should be fine.

[XhmikosR]

This is ready for review. The browserslist config I used is added in package.json. I can add the needed tools to automate this if you guys want later (postcss-cli and autoprefixer)

[XhmikosR]

Alright, I went ahead and added everything. Now everything is automated regarding CSS and JS, including CSS prefixes for our CSS but we still need to run autoprefixer (npm run prefix) manually.

[XhmikosR]

This is ready for review.

[XhmikosR]

Rebased

[PromoFaux]

@XhmikosR Is this one ready for review and merge?

[XhmikosR]

I think I need to do a couple more tests.

[XhmikosR]

@PromoFaux : I think this is finally ready. I split the patches a bit, tighten the browserslist settings and it should be good to go after one more tests round :)

[pralor-bot]

This pull request has been mentioned on Pi-hole Userspace. There might be relevant details there:

https://discourse.pi-hole.net/t/pi-hole-5-1-released/35577/1