ICON changeover completed, tests ok.
This commit is contained in:
@@ -3,22 +3,6 @@
|
||||
@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@
|
||||
|
||||
|
||||
todo: chunk-vendors has fa - brands in it which is fucking huge, what else is in there I don't need?
|
||||
eradicate fa - brands!
|
||||
Looks like I'm doing it wrong, I was using a generic NPM module when there is a Vue specific one that also has guidance to limit to just the icons I actually use:
|
||||
https://github.com/FortAwesome/vue-fontawesome
|
||||
This could be a big breaking change so need to do it carefully and all at once as it's own project
|
||||
Definitely need to do though, it will save a lot of size to just include the actual icons I'm using which is a handful
|
||||
|
||||
GUIDANCE
|
||||
See c:\temp\mytest project which uses the correct lib
|
||||
https://github.com/FortAwesome/vue-fontawesome#learn-about-our-new-svg-implementation
|
||||
https://stackoverflow.com/questions/52030435/fontawesome-with-vuetify-how-to-include-font-awesome-icons-within-the-v-icon-c/58261449#58261449
|
||||
|
||||
|
||||
todo: Test file associations in attach icon correct after above
|
||||
|
||||
|
||||
todo: login button can be obscured by footer, this should not happen try forcing margin below, footer maybe has a setting to confine it?
|
||||
move the login button?
|
||||
make sure enter works with it
|
||||
@@ -31,6 +15,7 @@ todo: chrome "Lighthouse" test against devops and implement recommendations
|
||||
### SMALL QUICK ITEMS #####
|
||||
todo: change logos from png to webp
|
||||
check if webp is supported e2e and on report
|
||||
|
||||
todo: login page seems to fetch both logos even though only one is visible
|
||||
maybe there is a v-show instead of a v-if or something?
|
||||
|
||||
|
||||
@@ -1,4 +1,4 @@
|
||||
export default {
|
||||
version: "8.0.0-alpha.63",
|
||||
version: "8.0.0-alpha.70",
|
||||
copyright: "© 1999-2020, Ground Zero Tech-Works Inc."
|
||||
};
|
||||
|
||||
Reference in New Issue
Block a user