search

replit / desktop

Replit Desktop App
114 stars 3 forks source link

Remove reload menu items from View menu in prod #119

Closed sergeichestakov closed 1 year ago

sergeichestakov commented 1 year ago

Why

Being able to "Refresh" a desktop app feels unnatural in some ways and makes it feel more like a web / electron app. It would also be nice to free up the Ctrl/Cmd + R keybinding for other use cases like running which would be useful for those coming from IDEs like xcode which has that same binding. See Slack thread.

Still keeping it around in "dev" (e.g. unpackaged) bc it would be useful to refresh the app in certain cases when developing or in a bad state.

What changed

Remove reload and force reload menu items from the View menu in prod

Test plan

Should not see reload/force reload after the next release. Ctrl/Cmd + R should not do anything and should be an available user-space keybinding.

sergeichestakov commented 1 year ago

yeah we're not planning on doing so in the short term. but honestly I don't think these decisions should be coupled since you can always "open in a new tab" and use the browser devtools so I don't consider a strict requirement.