Ticket #4848 (closed defect: fixed)

Opened 3 years ago

Last modified 2 weeks ago

[Patch] Previewing the edited section shows the whole page, instead of showing the preview of just the edited section.

Reported by: markell Assigned to: cbalan
Priority: high Component: SectionEditPlugin
Severity: major Keywords:
Cc: Trac Release: 0.11

Description

I am editing a section using the edit button enabled by the plugin. When I wish to preview my changes, the previewer displays the whole page, instead of just the section being edited. This is clearly a bug, because it effectively terminates the usefulness of the feature.

Attachments

preview-section.patch (1.3 kB) - added by mkc on 05/16/09 11:45:56.
Preview Page.png (3.1 kB) - added by rjollos on 12/01/09 01:47:36.
Preview Page.2.png (3.1 kB) - added by rjollos on 12/01/09 01:48:19.

Change History

03/28/09 21:36:55 changed by markell

  • summary changed from Previewing the edited section shows the whole page, instead showing just the preview of the just the edited section. to Previewing the edited section shows the whole page, instead showing the preview of just the edited section..

03/28/09 21:37:18 changed by markell

  • summary changed from Previewing the edited section shows the whole page, instead showing the preview of just the edited section. to Previewing the edited section shows the whole page, instead of showing the preview of just the edited section..

05/16/09 11:45:56 changed by mkc

  • attachment preview-section.patch added.

05/16/09 11:48:12 changed by mkc

Attached patch which adds "preview_whole_page" setting. When set to false, the preview is restricted to the section being modified only.

05/20/09 10:11:59 changed by rjollos

  • cc set to ryano@physiosonics.com.

05/21/09 10:17:45 changed by markell

Thanks mkc. Works great, although I would suggest (and that's what I actually did on my trac install) to change the default of preview_whole_page from True to False, since it makes more sense to preview just the section when editing just the section. Are you going to merge the change in the trunk for the plugin? Because currently it is not there. Thanks, again.

06/24/09 17:47:59 changed by anonymous

The patch works and really enhances usability. Sadly, the Image macro does not work in the section preview. The error is

sequence item 0: expected string, NoneType found

06/30/09 00:36:03 changed by rjollos

I can confirm the same problem with the Image macro.

06/30/09 00:36:14 changed by rjollos

  • cc changed from ryano@physiosonics.com to rjollos.

10/30/09 19:39:19 changed by rjollos

  • summary changed from Previewing the edited section shows the whole page, instead of showing the preview of just the edited section. to [Patch] Previewing the edited section shows the whole page, instead of showing the preview of just the edited section..

Hello cbalan,

Would it be possible to have this integrated into the repository? I'd like to provide some additional fixes for this plugin and it will be more straightforward than to have multiple patches that are dependent on one another.

(follow-up: ↓ 11 ) 10/30/09 19:39:38 changed by rjollos

One issue with the patch is that it sometimes causes the last line in the section to be replicated.

For instance, suppose my section is composed of:

line1
line2
line3
line4

After several edits and saves, I end up with something like:

line1
line2
line3
line4
line4
line4

However, the number of times the line is replicated doesn't seem to directly correspond to the number of saved revisions.

(in reply to: ↑ 10 ) 11/10/09 07:18:20 changed by rjollos

Replying to rjollos:

One issue with the patch is that it sometimes causes the last line in the section to be replicated.

Someone reported this behavior in #6169, so it may be unrelated to this patch.

(follow-up: ↓ 13 ) 11/10/09 10:17:27 changed by cbalan

(In [7117]) Applied preview whole page patch. Thank you mkc, rjolls. Refs #4848

(in reply to: ↑ 12 ; follow-ups: ↓ 14 ↓ 15 ↓ 16 ) 11/10/09 10:35:56 changed by cbalan

  • status changed from new to assigned.

Replying to cbalan:

(In [7117]) Applied preview whole page patch. Thank you mkc, rjolls. Refs #4848

Hey Guys,

What do you think about moving the preview_whole_page option to wiki edit page ? In this way, the user could switch the preview whole page on and off while editing/previewing

Thank you, Catalin Balan

(in reply to: ↑ 13 ) 11/10/09 10:37:55 changed by rjollos

  • cc deleted.

Replying to cbalan: What do you think about moving the preview_whole_page option to wiki edit page ? In this way, the user could switch the preview whole page on and off while editing/previewing

I'd find that feature to be useful.

(in reply to: ↑ 13 ) 11/11/09 22:25:42 changed by rjollos

Replying to cbalan:

What do you think about moving the preview_whole_page option to wiki edit page ? In this way, the user could switch the preview whole page on and off while editing/previewing

Would it be alright with you if we closed this ticket and opened another for this additional feature? That way it will be obvious to anyone that the patch has been incorporated into the trunk.

(in reply to: ↑ 13 ; follow-up: ↓ 18 ) 12/01/09 01:47:03 changed by rjollos

Replying to cbalan:

What do you think about moving the preview_whole_page option to wiki edit page ? In this way, the user could switch the preview whole page on and off while editing/previewing

I've been thinking about this. It might be better to add a Preview Section button next to the Preview Page button, rather than having a checkbox to toggle preview section / page.

Currently we have:

We could have:

Preview Page | Preview Section | Review Changes

12/01/09 01:47:36 changed by rjollos

  • attachment Preview Page.png added.

12/01/09 01:48:19 changed by rjollos

  • attachment Preview Page.2.png added.

12/01/09 01:48:46 changed by rjollos

Well, image didn't seem to upload correctly, but I'm sure you get the idea.

(in reply to: ↑ 16 ) 01/27/12 07:19:25 changed by rjollos

  • status changed from assigned to closed.
  • resolution set to fixed.

Replying to rjollos:

I've been thinking about this. It might be better to add a Preview Section button next to the Preview Page button, rather than having a checkbox to toggle preview section / page.

The preview page button seems like feature bloat. If the user chooses to edit a section, they should only see that section when previewing.

Add/Change #4848 ([Patch] Previewing the edited section shows the whole page, instead of showing the preview of just the edited section.)




Change Properties
Action