RoamJS / workbench

https://roamjs.com/extensions/workbench
The Unlicense
286 stars 35 forks source link

Migrate relevant settings to settings API #402

Open mdroidian opened 1 year ago

mdroidian commented 1 year ago

Reason

Better user experience.

Examples

Command Palette Plus

To remove the confirmation modal, create a block somewhere in your graph with the text: #42Setting workBenchDcpConfirm off

Daily Note Popup

To toggle this feature off, create a block somewhere in your graph with the text: #42Setting dailySubtitles off and refresh Roam.

Live Preview

You can edit the delay time for when the Live Preview appears. To do so, add the following block text to any block in your graph:

#42Setting LivePreviewDelay 100

etc

dvargas92495 commented 1 year ago

Yup - this was all in the "I'll deal with it after it's in Roam Depot" bucket. Adding to backlog

mdroidian commented 1 year ago

With all of the feature/modules, the Roam settings page is already quite long. This requires the user to scroll to get to any settings. Not to mention all the hotkeys that would be added when Command Palette+ / HotKeys is toggled.

I'm exploring different options to handle this.

Remove Description from Feature Toggles

Don't really like this option. Doesn't seem to reduce enough space. User would have to know to scroll to find descriptions. image

Custom React Table

Better. But the "Name" of the setting takes up a lot of space image

Settings in page

I'm kind of a fan of what's been done with roam/js/weekly-notes image

We could update the settings descriptions to link to these pages

mdroidian commented 1 year ago

For context, here are the settings for each feature/module I noted

image