Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
* Reformatted .inc and our packackage.json to have consistent 2 spaces of padding * changed all var's to const/let * removed extended data as unused * removed leading zeros from scss files. This is more consistent with the style used across most of the project * removed object shorthands. This way is more consistent across the existing codebase * switched from the rgba to the rgb css widget. Reasoning is, that it seems a bit more readable and browser support is there: https://caniuse.com/mdn-css_types_color_rgb_alpha_parameter * removed useless extra clutter in the scss files, which stylint wants * whitespace changes for js files * removed .scss from scss imports for consistency * Changed aditional var to const/let * Removed curly string and console warnings, as they are more noise than signal * fixed non scss compliant use of rgb added stylelint-scss to the plugins * changes to make scss files compliant: - modules should be used instead of raw global functions - `border: 0;` is more constinent to `border: none;` - Only let stylint run over css files (it has some isues with the vue syntax) * removed the unused function GETRequest() * history=>window.history * Used Object.values instead of for..in Reason: for..in loops iterate over the entire prototype chain, which is virtually never what you want. Use Object.{keys,values,entries}, and iterate over the resulting array * added globals for the common js globals * allowed dangling underscored and made more things contain underscores * fixed how for loops are used: .forEach is superiour to for..of due to Polyfills and other issues * moved addEventListener to document.addEventListener * prefered explicit function-arg defaults, instead of exchanging after the fact * moved regenerator-runtime to npm * removed CustomEvent polyfill (for IE) * mostly unified the functions (named,unnamed, arrow) used by JS gulpfile.js is the odd one out. * migrated from node-sass to dartSass. Reasoning: sass/node-sass#2886 * reordered the dependencies in the package.json: - only things that modern browsers use are getting delivered are listed as dependencys. - polyfills/build tooling is listed as below the other Dependencies, because while included in prod, they are only delivered to a minority of users * replaces navigatum.cloneState with the buildin structuredClone see https://caniuse.com/mdn-api_structuredclone with a polyfill for samsung internet * added conditional feedback.js minification * changed the method how we rename files in the gulpfile * added better maintianed version of spectre.css Co-authored-by: octycs <[email protected]> Co-authored-by: octycs <[email protected]>
- Loading branch information
3 people
authored
Jul 31, 2022
1 parent
1f18d27
commit 6e16862