Modify

Opened 12 years ago

Last modified 12 years ago

#7190 new defect

allow disabling of highlighting/cursor changes

Reported by: NE2 Owned by: team
Priority: normal Milestone:
Component: Core Version:
Keywords: Cc:

Description

I may be wrong, but this feature seems to have caused major sluggishness for me. Is there a way to turn it off? If not, can you please add one? Thanks.

Attachments (1)

DrawingOptions.patch (7.3 KB ) - added by akks 12 years ago.

Download all attachments as: .zip

Change History (22)

comment:1 by akks, 12 years ago

It would be really good for slow machines or large datasets.

comment:2 by akks, 12 years ago

Should I make the patch or someone already have such plans?

comment:3 by bastiK, 12 years ago

Go ahead, maybe we can group settings, that affect performance. (rubber line, antialiasing, highlight, ...)

comment:4 by akks, 12 years ago

Luckily, there was ready option "draw.target-highlight" for highlighting. I have just added it to configuration panel and grouped entries about highlighting, rubber line, antialiasing and area fill together.

Here is a patch:

by akks, 12 years ago

Attachment: DrawingOptions.patch added

comment:5 by akks, 12 years ago

Summary: allow disabling of highlighting/cursor changes(patch) allow disabling of highlighting/cursor changes

comment:6 by stoecker, 12 years ago

Most of these options should be expert options. Can you change that?

comment:7 by bastiK, 12 years ago

Summary: (patch) allow disabling of highlighting/cursor changes[patch] allow disabling of highlighting/cursor changes

comment:8 by akks, 12 years ago

Of course I can if you give me list of options that should be "expert" :)

I do not know. For example, hiding rubber line and disabling highlighting really improve performance and may be needed for non-expert users too...

comment:9 by stoecker, 12 years ago

Choose yourself :-)

Everything which is not really essential, e.g. most of this visual stuff is "expert". Aim of the normal mode is to have a software which only presents essential options to the user and thus reducing the initial amount of possibilities.

Every option in the preferences has a good reason, sure. Otherwise they wouldn't be there. But as a beginner it is much harder to find 5 really important options when they are hidden between 50 "nice-to-have" options.

comment:10 by akks, 12 years ago

OK, but it will not be fast then :)
If anyone have ideas about specific really important options, please write it here...

comment:11 by skyper, 12 years ago

Thanks for your work akks.

I am often working with slow machines and I am happy about every Mb RAM I can save. I have to admit though that at least in the first world people with slow machines will probably also find the expert options. This might be totally different in other regions.

Maybe some start option would be nice to run JOSM on slow machines with these features turn off. Another solution would be to do it automatically depending on the given RAM size.

comment:12 by akks, 12 years ago

I have moved some prefrerence to expert mode, commiting now... I'll leave ticket open for now, maybe you notice some problems or can implement more echancements.

comment:13 by akks, 12 years ago

In [4767/josm]:

Option "Highlight target ways and nodes" in preferences, see #7190

comment:14 by NE2, 12 years ago

Thanks a lot. This seems to run much better.

comment:15 by bastiK, 12 years ago

close?

in reply to:  12 ; comment:16 by skyper, 12 years ago

Replying to akks:

I have moved some prefrerence to expert mode, commiting now... I'll leave ticket open for now, maybe you notice some problems or can implement more echancements.

Is "show object ids" really an expert option ?

I still miss the option to disable the changes of the cursor. At least this is what the summary says.

comment:17 by skyper, 12 years ago

Summary: [patch] allow disabling of highlighting/cursor changesallow disabling of highlighting/cursor changes

in reply to:  16 comment:18 by akks, 12 years ago

Replying to skyper:

Is "show object ids" really an expert option ?

I still miss the option to disable the changes of the cursor. At least this is what the summary says.

It was not in my [4767] :) I think this option shoudld not be expert, of course.

Last edited 12 years ago by akks (previous) (diff)

in reply to:  16 ; comment:19 by bastiK, 12 years ago

Replying to skyper:

I still miss the option to disable the changes of the cursor.

Why? This won't improve the performance.

in reply to:  19 comment:20 by skyper, 12 years ago

Replying to bastiK:

Replying to skyper:

I still miss the option to disable the changes of the cursor.

Why? This won't improve the performance.

Then it is stupid. Please forget it.

comment:21 by akks, 12 years ago

Actually, most CPU-intense task is rendering (painting). Highligting (and sometimes rubber band) caused all the map to repaint after each mouse movement.

Next, less hard task is searching primitives under the cursor after mouse move (to change cursors, show info in statusbar, change highlighting). It is integrated with instruments and can not be easily turned off... But if it is needed, this task can also become optional.

Version 0, edited 12 years ago by akks (next)

Modify Ticket

Change Properties
Set your email in Preferences
Action
as new The owner will remain team.
as The resolution will be set. Next status will be 'closed'.
to The owner will be changed from team to the specified user.
Next status will be 'needinfo'. The owner will be changed from team to NE2.
as duplicate The resolution will be set to duplicate. Next status will be 'closed'. The specified ticket will be cross-referenced with this ticket.
The owner will be changed from team to anonymous. Next status will be 'assigned'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.