[Geoserver-devel] One final tweak to SLD autocomplete before RC?

Hi,
I’ve been playing with the SLD editor and its autocomplete support for a while now,
the current behavior is … annoying? :slight_smile:
When just madly typing down a style it works fine, but when tweaking an existing style, it’s sometimes maddening.

For example:

  • I’m going around with the arrows and it randomly shows auto completion that would be suitable for that location (not at every keystroke, but often enough in an editing session, try editing bits of an existing style and then go around)

  • Maybe I’m trying to reformat the style a bit and again it shows up breaking the editing flow

  • Also, when creating a new style, the editors open empty, and it’s already offering to start with StyledLayerDescriptor, with the editor not even yet in focus (try typing down the style name and then tab your way to the editor, and there is happens)
    Could we go back to activating the completion using CTRL-SPACE instead? :slight_smile:

Cheers
Andrea

···

GeoServer Professional Services from the experts! Visit http://goo.gl/it488V for more information. == Ing. Andrea Aime @geowolf Technical Lead GeoSolutions S.A.S. Via di Montramito 3/A 55054 Massarosa (LU) phone: +39 0584 962313 fax: +39 0584 1660272 mob: +39 339 8844549 http://www.geo-solutions.it 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.

Collecting last moment feedback like this is important, by all means make it less annoying.

aside: We really did not get any feedback from the milestone release did we?

···


Jody Garnett

Discussed in the PMC meeting, there is now a PR for this change here: https://github.com/geoserver/geoserver/pull/3335

Cheers,

Torben

On Tue, Jan 22, 2019 at 8:28 AM Andrea Aime <andrea.aime@anonymised.com> wrote:

Hi,
I’ve been playing with the SLD editor and its autocomplete support for a while now,
the current behavior is … annoying? :slight_smile:
When just madly typing down a style it works fine, but when tweaking an existing style, it’s sometimes maddening.

For example:

  • I’m going around with the arrows and it randomly shows auto completion that would be suitable for that location (not at every keystroke, but often enough in an editing session, try editing bits of an existing style and then go around)

  • Maybe I’m trying to reformat the style a bit and again it shows up breaking the editing flow

  • Also, when creating a new style, the editors open empty, and it’s already offering to start with StyledLayerDescriptor, with the editor not even yet in focus (try typing down the style name and then tab your way to the editor, and there is happens)
    Could we go back to activating the completion using CTRL-SPACE instead? :slight_smile:

Cheers
Andrea

==

GeoServer Professional Services from the experts! Visit http://goo.gl/it488V for more information. == Ing. Andrea Aime @geowolf Technical Lead GeoSolutions S.A.S. Via di Montramito 3/A 55054 Massarosa (LU) phone: +39 0584 962313 fax: +39 0584 1660272 mob: +39 339 8844549 http://www.geo-solutions.it 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

There was the Marlin one, but it would require some changes in the linux/win command
shells to recognize java 11 and don’t force our own Marlin version. I think I’ve seen some
snippets for bash on StackOverflow, but I’ve got nothing for Windows.

We could also update Marlin to the latest release while we’re at it.

Cheers
Andrea

···

GeoServer Professional Services from the experts! Visit http://goo.gl/it488V for more information. == Ing. Andrea Aime @geowolf Technical Lead GeoSolutions S.A.S. Via di Montramito 3/A 55054 Massarosa (LU) phone: +39 0584 962313 fax: +39 0584 1660272 mob: +39 339 8844549 http://www.geo-solutions.it 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.

Blah, that feedback was based on a screen snap on twitter.

I guess I will look at the windows one then - looks like there are some examples to start from.

On Wed, 23 Jan 2019 at 04:22, Andrea Aime <andrea.aime@anonymised.com> wrote:

On Wed, Jan 23, 2019 at 12:58 AM Jody Garnett <jody.garnett@anonymised.com> wrote:

Collecting last moment feedback like this is important, by all means make it less annoying.

aside: We really did not get any feedback from the milestone release did we?

There was the Marlin one, but it would require some changes in the linux/win command
shells to recognize java 11 and don’t force our own Marlin version. I think I’ve seen some
snippets for bash on StackOverflow, but I’ve got nothing for Windows.

We could also update Marlin to the latest release while we’re at it.

Cheers
Andrea

==

GeoServer Professional Services from the experts! Visit http://goo.gl/it488V for more information. == Ing. Andrea Aime @geowolf Technical Lead GeoSolutions S.A.S. Via di Montramito 3/A 55054 Massarosa (LU) phone: +39 0584 962313 fax: +39 0584 1660272 mob: +39 339 8844549 http://www.geo-solutions.it 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.