iD icon indicating copy to clipboard operation
iD copied to clipboard

Release as Electron app

Open verhovsky opened this issue 1 year ago • 4 comments

Packaging iD as an Electron application has two benefits I can think of (besides the obvious)

  1. swiping back with two fingers on a touchpad shouldn't act as a back button like it does on macos. i've lost data because of this
  2. you can have more undo history because localsessionstorage is 50MB instead of 10MB. you could also write to disk and get effectively unlimited history

verhovsky avatar Mar 15 '24 21:03 verhovsky

  1. This annoying behaviour can be disabled in settings for all apps, or just for chrome
  2. We could easily fix this by using IndexedDB instead of localStorage, then there would be effectively no size limit†. Using IndexedDB would also improve performance, since we no longer need to call JSON.stringify on an a huge object everytime you drag a node. Incidentally, that's one of the biggest performance bottlenecks that I've noticed in iD

k-yle avatar Mar 16 '24 01:03 k-yle

I'm not going to disable it for Chrome because I like it on all websites except iD.

verhovsky avatar Mar 16 '24 01:03 verhovsky

(besides the obvious)

Which one is obvious for you?

matkoniecz avatar Apr 05 '24 03:04 matkoniecz

less taps to open, less data to load

verhovsky avatar Apr 05 '24 04:04 verhovsky

At the current point, i think it would be too much effort to maintain this for the relatively little benefits it provides.

tyrasd avatar Apr 08 '25 08:04 tyrasd