-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Panels need refresh buttons #133
Comments
We should also have a refresh button on the left panel as well. This is useful during development when the developer may want to iterate constantly on the fields/schema setup. Currently the only way to reload the fields is to refresh the whole page. |
NOTE: Reload button for the fields panel might be useful for non-devs too. It can serve as a way to "discard changes" and reload whatever was from GitHub. |
I've updated the mocks with a refresh icon in the editor panel header, near the localization and panel expansion icons: Note: this mock current replaces the environment links in the previous iterations (see screenshot below), however I noticed this feature is no longer included in the current version of the app. If we're still looking to support this idea in the future, then I will revise the mocks further. Please let me know. |
Right-side refresh buttons look great to me. Let's add them as soon as we can since they'll be really useful during the development phase of building out Editor sites. Regarding the link buttons - we can have a deeper discussion around those. I'm wondering if we may want to put those links inside an action menu (and therefore use text links instead). For me personally it is hard to remember what each icon means. |
I'm really curious how much you will end up using this as I usually find it a lot faster to switch to the browser and use How much you will use it if you have to use the mouse to navigate to refresh the settings vs refreshing the whole page. Especially when you are doing it against a local code base since it doesn't take long for the refresh to happen. |
Or a better way of saying that: we should have some keyboard shortcut that makes this process easier than having to click the icon for power users who don't want to refresh the entire app. |
Instead of needing to reload the entire page to check for changes it now reloads just the individual file and clears the partials. Also updates the states for amagaki and grow to have the `partialsOrGetPartials` style of accessing the partials to prevent the need for the UI to keep track of the initial load. part of #133
@micjamking Could we get your opinion on where to place it?
The text was updated successfully, but these errors were encountered: