Hi Jody, I've just skimmed through the proposal. The idea of modernization and cleanup is nice, and needed. The backwards compatibility breakage on custom CSS for the GeoServer UI less so, but I think they are not used very much, not particularly worried about it.
What gets me really worried though is that this is not just a geoserver.css change, but a wholesale change in all HTML files to follow it: - I don't see any visual indication in the proposal, does it mean the UI will look exactly the same? If not, do you already know what you're going to change? - With just two developers, I don't see how you can go over all extensions and community modules. If you simply intend to break all community modules, you should say so, it should be a prominent message in the proposal (they are not supported, but it does not mean they are not used, PSC will have to agree to take on the upgrade work that will follow, as it happened for the REST config API sprint for example). A reasonable approach would be to ensure old GUI modules do gracefully work even with the new CSS, and would also help cover customizations/plugins outside of the repository (plenty of them around). - Even limiting yourself to core and extensions, are you going to manually test before/after all the extension modules? Are you familiar with all of them? - Going through the HTML files you'll have the temptation to just add other unrelated changes to "make things better".. but these are not really covered by the proposal. Can you resist the temptation? Cheers Andrea On Wed, Jun 15, 2022 at 3:25 PM Jody Garnett <jody.garn...@gmail.com> wrote: > Jody and Michel here from the Bolsena code sprint with a proposal for > cleaning up the use of css: > > https://github.com/geoserver/geoserver/wiki/GSIP-213 > > Scroll down to the bottom of the proposal for an example, feedback (and > encouragement) welcome. > -- > Jody Garnett > _______________________________________________ > Geoserver-devel mailing list > Geoserver-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/geoserver-devel > -- Regards, Andrea Aime == GeoServer Professional Services from the experts! Visit http://bit.ly/gs-services-us for more information. == Ing. Andrea Aime @geowolf Technical Lead GeoSolutions Group phone: +39 0584 962313 fax: +39 0584 1660272 mob: +39 333 8128928 https://www.geosolutionsgroup.com/ http://twitter.com/geosolutions_it ------------------------------------------------------- Con riferimento alla normativa sul trattamento dei dati personali (Reg. UE 2016/679 - Regolamento generale sulla protezione dei dati “GDPR”), si precisa che ogni circostanza inerente alla presente email (il suo contenuto, gli eventuali allegati, etc.) è un dato la cui conoscenza è riservata al/i solo/i destinatario/i indicati dallo scrivente. Se il messaggio Le è giunto per errore, è tenuta/o a cancellarlo, ogni altra operazione è illecita. Le sarei comunque grato se potesse darmene notizia. This email is intended only for the person or entity to which it is addressed and may contain information that is privileged, confidential or otherwise protected from disclosure. We remind that - as provided by European Regulation 2016/679 “GDPR” - copying, dissemination or use of this e-mail or the information herein by anyone other than the intended recipient is prohibited. If you have received this email by mistake, please notify us immediately by telephone or e-mail
_______________________________________________ Geoserver-devel mailing list Geoserver-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/geoserver-devel
