The 2026 Drupal Business Survey is open. Your response will help shape the future of the Drupal business ecosystem.The current problem with the implementation of the preview in d7 ui is that
the view is runned on the initial php loading. This leads to a lot of problems
- If the sql breaks you might have a problem
- Performance: you have to execute a query just to edit it isn't that perfect
So that the preview is additional, it needs some kind of seperate page callback to support the non-js use case.
| Comment | File | Size | Author |
|---|---|---|---|
| #7 | 1257376-preview.patch | 2.84 KB | dawehner |
| #3 | 1257376-preview.patch | 2.83 KB | dawehner |
| #1 | 1257376-preview.patch | 2.36 KB | dawehner |
Comments
Comment #1
dawehnerHere is an initial patch which tries to bring this back.
One feature which this drops at the moment is that on the initial loading the preview is done.
Comment #2
dawehnerUpdate status.
Comment #3
dawehnerHere is some more work. One problem is that the checkbox is pretty much ignored if you edit some kind of setting.
It always rerenders.
Comment #4
webchickTagging as a release blocker, per dereine.
Comment #5
tim.plunkettI'm not sure which checkbox you mean. It seems to behave as I expect it to.
Comment #6
dawehnerI mean the "[] auto preview" checkbox.
If you change something and the auto preview checkbox is not checked, it shouldn't run the preview, but it does it at the moment.
Comment #7
dawehnerJust remembered that checkboxes are special in javascript :( you can't just use val() but you have to use is(':checked').
Comment #8
tim.plunkettIf the checkbox on admin/structure/views/settings for "Automatically update preview on changes" is checked, the view is still previewed on page load and this change has no effect.
If that checkbox is unchecked, then the patch works as advertised.
I *think* this is the desired effect, and it's what the patch documentation says, so RTBC from me.
Comment #9
dawehnerThanks for the review! Commited to 7.x-3.x