treesheets icon indicating copy to clipboard operation
treesheets copied to clipboard

Exporting HTML only exports zoomed cells

Open gussmith23 opened this issue 7 years ago • 5 comments

This may be an intentional feature.

When I zoom in on a cell using ctrl+mousewheel and then save, an HTML representation gets exported as usual, but ONLY of the zoomed-in cells. This is worrying because I use the HTML export feature as a backup/for record-keeping. If I zoom in, make an edit, save, and exit TreeSheets, I will have now overwritten the HTML file with a more up-to-date version of the sheet which is also a zoomed-in version of the sheet, potentially leaving out a bunch of data.

gussmith23 avatar Feb 20 '18 22:02 gussmith23

For HTML export using the export menu, that is intended. If this is also happening during automatic HTML export on every normal save, that is not intended/desired.

aardappel avatar Feb 24 '18 16:02 aardappel

Yes -- it's happening on save, too.

On Sat, Feb 24, 2018, 11:21 AM Wouter van Oortmerssen < [email protected]> wrote:

For HTML export using the export menu, that is intended. If this is also happening during automatic HTML export on every normal save, that is not intended/desired.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/aardappel/treesheets/issues/100#issuecomment-368239597, or mute the thread https://github.com/notifications/unsubscribe-auth/ABJcgR-gBT8deASrTBphzMJj3qbR2FWOks5tYDcVgaJpZM4SMv9j .

gussmith23 avatar Feb 24 '18 16:02 gussmith23

Ok, just checked, and when zoomed in it still exports the whole file as HTML. Checked the code and it explicitly selects the root when auto-exporting. So not sure whats going on in your case.

aardappel avatar Feb 24 '18 16:02 aardappel

Hm...I just tested it again on my Windows system, and this is the behavior of the different export methods:

  1. Exporting from File-->Export view as: exports just the zoomed-in cells (this is what it's supposed to do)
  2. Auto-exporting on every save: also only exports the zoomed-in cells (this is not what it's supposed to do, apparently, and it's also apparently not happening on your system?)

Perhaps this is just a version issue. At some point i can try to build the latest version and see if this is still an issue.

On Sat, Feb 24, 2018 at 11:49 AM, Wouter van Oortmerssen < [email protected]> wrote:

Ok, just checked, and when zoomed in it still exports the whole file as HTML. Checked the code and it explicitly selects the root when auto-exporting. So not sure whats going on in your case.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/aardappel/treesheets/issues/100#issuecomment-368241611, or mute the thread https://github.com/notifications/unsubscribe-auth/ABJcgQNWlMh1D8LmKQEFZ-DzEbBWe6TGks5tYD2FgaJpZM4SMv9j .

-- Gus Smith Penn State University M.S./B.S. Computer Science and Engineering '18

Microsystems Design Lab - Researcher

(570)817-9340 | [email protected]

gussmith23 avatar Feb 25 '18 16:02 gussmith23

Yup tested on Windows too (though that shouldn't matter). That code also hasn't changed in forever, so not sure how it could be a version issue.

If you can build it in Visual Studio and are able to reproduce it, you could set a breakpoint on the first line of ExportFile() (currently document.h:656) and inspect the values of curdrawroot / rootgrid to see what's going on.

aardappel avatar Feb 25 '18 17:02 aardappel