Skip to content
This repository has been archived by the owner on Jul 24, 2021. It is now read-only.

[PATCH] Move to the next field on enter in properties dock tags #1137

Closed
openstreetmap-trac opened this issue Jul 23, 2021 · 6 comments
Closed

Comments

@openstreetmap-trac
Copy link

Reporter: tcarter[at]noggin.com.au
[Submitted to the original trac issue database at 10.29am, Saturday, 30th August 2008]

The attached patch implements move to next field behavior when pressing enter/return when editing tags, this makes it easier to enter multiple tags without using the mouse.

@openstreetmap-trac
Copy link
Author

Author: koying
[Added to the original trac issue at 11.15pm, Tuesday, 2nd September 2008]

Thanks for all your patches.

On this particular one, I'd like the escape key to just close / de-focus the current delegate tag edit, rather than the key to be passed to the main window, de-selecting the edited feature.

Do you think it is possible?

@openstreetmap-trac
Copy link
Author

Author: tcarter[at]noggin.com.au
[Added to the original trac issue at 1.16am, Wednesday, 3rd September 2008]

I would guess it should be possible, and that's I was trying for originally, but I haven't figured out how to override the global key binding for an individual widget yet.

It's something I'm planning to try to figure out though, I was also planning to bind F2 to start editing with the cursor at the end of the current tag name/value (eg like the standard rename shortcut), but it's globally bound to zoom all - do you think it would be appropriate to override that in the tag edit too?

@openstreetmap-trac
Copy link
Author

Author: koying
[Added to the original trac issue at 9.36am, Wednesday, 3rd September 2008]

(In [10442]) ADD : Support for GeoTagged images. qmake GEOIMAGE=1 to activate. requires exiv2 (by Timo Schl?\195?\188?\195?\159ler)
ADD : Move to the next field on enter in properties dock tags (by Travers Carter) (Refs #1137)

@openstreetmap-trac
Copy link
Author

Author: Trav
[Added to the original trac issue at 1.22am, Sunday, 14th September 2008]

I've updated the keyboard interactions

  • Escape now closes the edit delegate combobox without committing, but doesn't de-select
  • Cursor keys now navigate through the table cells
  • F2 starts an edit without overwriting the current cell's contents
  • Delete starts an edit clearing the current cell's contents
  • Tab/Backtab now only cycles through tag names/values when an edit delegate is open,
    I think this is better because you can Tab/Backtab to the other Docks and widgets
    more easily, but just ignore the .ui changes if you disagree)
  • Applied the width & nav fix to the MultiUI tagview too.

The attached patch (merkaartor-tagview-nav2.diff) is against the current SVN

@openstreetmap-trac
Copy link
Author

Author: koying
[Added to the original trac issue at 11.33am, Saturday, 20th September 2008]

Thanks!

@openstreetmap-trac
Copy link
Author

Author: koying
[Added to the original trac issue at 11.37am, Saturday, 20th September 2008]

(In [10801]) ADD : Move to the next field on enter in properties dock tags - part 2 (by Travers Carter) (closes #1137)
FIX : compile error on GCC

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant