This project is currently using sanitize.css which is not updated for 3 years.
overflow-wrap: break-word too offensive but it's ok.
Points against using sanitize.css:
Unnecessary declarations that opinionated and not needed:
-
background-repeat: no-repeat;, no need to set this, it should be using default behavior which is repeat to be more predictable.
-
text-decoration: inherit; text-decoration property used in those elements: a, s, u, del, ins, strike and there is no need to set the inheritance over ::before, ::after, same for vertical-align: inherit.
-
cursor: default in most elements cursor: default is a default value and another things like cursor: auto for input, textarea, so it's no need to set up, let the browser do its thing.
-
In abbr[title] there is 2 declarations indeed duplicated: underline then underline dotted as values, underline dotted used by Firefox as default (so it should be used as only declaration):
abbr[title], acronym[title] {
text-decoration: dotted underline;
}
-
small font-size: 80% unneeded because it's font set it up 2 see: main.css = font-size: .8em;
-
margin: 0 to button, input, select elements: not needed because button, margin has no margin and input[type="*"] have some margins but not big numbers so something like Stack layout would be adorable and it wouldn't need us to set margin to 0. ( .stack>+{ margin-block-start: 1rem } )
-
-webkit-appearance: button should be avoided to use.
-
border: 1px solid #a0a0a0 set for fieldset to achieve "Change the inconsistent appearance in all browsers (opinionated)." but actually there is no inconsistency: see https://browserdefaultstyles.com/#fieldset default value is border: groove 2px ThreeDFace
-
margin: 0 to textarea: textarea has no margin by default: https://browserdefaultstyles.com/#textarea
-
input[type="search"] no need for those declarations because outline-offset is 0 by default see: https://browserdefaultstyles.com/#[type=search]
-
In fact vendor specific things should be avoided...
-
:focus-visible isn't needed but :target is fine but no need for z-index: 2
-
summary declaration is ok but it's better to use just summary and not details > summary:first-of-type because website authors themselves shouldn't add more then one summary to details element and for not increasing specificity it's better to stick with summary.
-
datalist { display: none } unnecessary it's default see: https://browserdefaultstyles.com/#datalist
This project is currently using sanitize.css which is not updated for 3 years.
overflow-wrap: break-wordtoo offensive but it's ok.Points against using sanitize.css:
Unnecessary declarations that opinionated and not needed:
background-repeat: no-repeat;, no need to set this, it should be using default behavior which isrepeatto be more predictable.text-decoration: inherit;text-decorationproperty used in those elements:a, s, u, del, ins, strikeand there is no need to set the inheritance over::before, ::after, same forvertical-align: inherit.cursor: defaultin most elementscursor: defaultis a default value and another things likecursor: autoforinput, textarea, so it's no need to set up, let the browser do its thing.In
abbr[title]there is 2 declarations indeed duplicated:underlinethenunderline dottedas values,underline dottedused by Firefox as default (so it should be used as only declaration):smallfont-size: 80%unneeded because it's font set it up 2 see:main.css=font-size: .8em;margin: 0tobutton, input, selectelements: not needed becausebutton, marginhas no margin andinput[type="*"]have some margins but not big numbers so something like Stack layout would be adorable and it wouldn't need us to set margin to 0. ( .stack>+{ margin-block-start: 1rem } )-webkit-appearance: buttonshould be avoided to use.border: 1px solid #a0a0a0set forfieldsetto achieve "Change the inconsistent appearance in all browsers (opinionated)." but actually there is no inconsistency: see https://browserdefaultstyles.com/#fieldset default value isborder: groove 2px ThreeDFacemargin: 0totextarea: textarea has no margin by default: https://browserdefaultstyles.com/#textareainput[type="search"]no need for those declarations because outline-offset is 0 by default see: https://browserdefaultstyles.com/#[type=search]In fact vendor specific things should be avoided...
:focus-visibleisn't needed but:targetis fine but no need forz-index: 2summarydeclaration is ok but it's better to use justsummaryand notdetails > summary:first-of-typebecause website authors themselves shouldn't add more then one summary to details element and for not increasing specificity it's better to stick withsummary.datalist { display: none }unnecessary it's default see: https://browserdefaultstyles.com/#datalist