Skip to content
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

PPTX export and settings page #11

Open
wants to merge 28 commits into
base: develop
Choose a base branch
from

Conversation

tung-jin-chew-hp
Copy link
Contributor

@tung-jin-chew-hp tung-jin-chew-hp commented Feb 21, 2017

These are the PPTX export changes; reduced to remove the dategraph and dashboard report logic, with export buttons for the visualizations, e.g.
topicmap.pptx
list.pptx
map.pptx
sunburst.pptx
table.pptx
comparisonmap.pptx

The composite report controller is also present; though that'll have to wait for the dashboard before any UI changes are present to make it usable; those are excluded from this pull request.
composite
e.g. report.pptx

There's a settings page to allow specifying a custom template, e.g. if you want to provide your own slide with a master slide configured to put your logo on every page. It also allows setting margins to reserve space for your custom content. There's validation of the template format when you save or press the 'Test Template' button.
settings

@tung-jin-chew-hp tung-jin-chew-hp force-pushed the feature/pptx branch 3 times, most recently from 5c9e876 to 760f131 Compare March 2, 2017 16:08
@tung-jin-chew-hp
Copy link
Contributor Author

Rebased onto develop, which now has the dashboard, so squashed the report export code in.

@tung-jin-chew-hp tung-jin-chew-hp force-pushed the feature/pptx branch 2 times, most recently from dcb1171 to fdf808d Compare March 3, 2017 18:37
@tung-jin-chew-hp
Copy link
Contributor Author

Rebased again for new sunburst

@tung-jin-chew-hp
Copy link
Contributor Author

tung-jin-chew-hp commented Mar 10, 2017

Rebased again to include the new Sunburst dashboard widget; and added PowerPoint export support for it.

@tung-jin-chew-hp tung-jin-chew-hp force-pushed the feature/pptx branch 9 times, most recently from d96de7b to ffb6aed Compare March 16, 2017 11:09
@tung-jin-chew-hp
Copy link
Contributor Author

More rebases, fixes to work with the rebased code, render the list widgets with images now that thumbnails are included on it as of FIND-971, timing improvements to work with the CasperJS export, misc other improvements.

…e of merging, removing all the merged dashboard and dategraph changes.

Squashed commit of the following:

commit edaa4c9
Author: Tung Jin Chew <[email protected]>
Date:   Fri Feb 17 18:48:24 2017 +0000

    Fix missing import in settings page.

commit 9a3305b
Author: Tung Jin Chew <[email protected]>
Date:   Fri Feb 17 11:31:42 2017 +0000

    Use new API signature

commit ead0659
Author: Tung Jin Chew <[email protected]>
Date:   Fri Feb 17 09:57:51 2017 +0000

    The nullable parameters are now last (though we'd like them in the UI, so we keep the @RequestParam as required)

commit 3095c18
Author: Tung Jin Chew <[email protected]>
Date:   Thu Feb 16 11:13:57 2017 +0000

    Correct margin-to-anchorpoint calculations

commit 271f7c8
Author: Tung Jin Chew <[email protected]>
Date:   Thu Feb 16 10:50:40 2017 +0000

    Use the official HPE accent colour

commit beb9a16
Author: Tung Jin Chew <[email protected]>
Date:   Wed Feb 15 19:52:28 2017 +0000

    Add an indicator to give some idea of what the margin percentages look like

commit 958fa8a
Author: Tung Jin Chew <[email protected]>
Date:   Wed Feb 15 15:09:18 2017 +0000

    Add i18n message for the invalid-margins error

commit 83a0606
Author: Tung Jin Chew <[email protected]>
Date:   Wed Feb 15 15:05:42 2017 +0000

    Add button and controller method to download the default sample template. Fix rendering in Firefox.

commit 02ebf45
Author: Tung Jin Chew <[email protected]>
Date:   Wed Feb 15 14:26:29 2017 +0000

    Add UI, logic and validation checking for top/bottom/left/right margins on the PowerPoint templates; which is useful if the user wants to reserve space for their own logos etc. on the default master slide.

commit 968581a
Author: Tung Jin Chew <[email protected]>
Date:   Tue Feb 14 19:27:58 2017 +0000

    Show the reason why it's invalid if the template is reported as invalid; hide the message if it's valid.

commit f699b2c
Author: Tung Jin Chew <[email protected]>
Date:   Tue Feb 14 19:11:16 2017 +0000

    Add a validate button to the powerpoint widget

commit bdb32ef
Author: Tung Jin Chew <[email protected]>
Date:   Tue Feb 14 18:53:40 2017 +0000

    Show error validation status on the powerpoint template input when saving

commit 3893e48
Author: Tung Jin Chew <[email protected]>
Date:   Tue Feb 14 17:24:28 2017 +0000

    Add save-time validation messages for the error when the PowerPoint template is invalid

commit 0909314
Author: Tung Jin Chew <[email protected]>
Date:   Tue Feb 14 16:59:40 2017 +0000

    Don't allow saving when the powerpoint is invalid

commit 5d405a9
Author: Tung Jin Chew <[email protected]>
Date:   Tue Feb 14 16:16:08 2017 +0000

    Allow user to specify a custom PowerPoint template file. Add UI for this in the settings page.

commit 4da3feb
Author: Tung Jin Chew <[email protected]>
Date:   Tue Feb 14 12:01:29 2017 +0000

    Fix tests

commit 8933438
Author: Tung Jin Chew <[email protected]>
Date:   Tue Feb 14 11:56:12 2017 +0000

    Cleanup: remove old TODO and reduce changes for simpler merge; delete file which has been superceded.

commit bdef27d
Author: Tung Jin Chew <[email protected]>
Date:   Tue Feb 14 11:30:33 2017 +0000

    Use the default autowired object mapper instead of creating a new instance on each request

commit 7e5b9ea
Author: Tung Jin Chew <[email protected]>
Date:   Tue Feb 14 11:30:08 2017 +0000

    Default template has been moved to the external package

commit dcc0a19
Author: Tung Jin Chew <[email protected]>
Date:   Mon Feb 13 20:09:10 2017 +0000

    Distinguish service from implementation

commit 1ca5331
Author: Tung Jin Chew <[email protected]>
Date:   Mon Feb 13 20:00:19 2017 +0000

    It seems it's necessary to include the reports package in directly; otherwise spring-boot doesn't track its transitive dependencies

commit 51a2a2f
Author: Tung Jin Chew <[email protected]>
Date:   Mon Feb 13 19:49:26 2017 +0000

    Separately package the powerpoint report-generation backend into its own Maven artifact.

commit f59b9fb
Author: Tung Jin Chew <[email protected]>
Date:   Mon Feb 13 18:22:08 2017 +0000

    Create a templatesource class so we won't have to include Spring core in our powerpoint package once it's split off

commit 03aadf0
Author: Tung Jin Chew <[email protected]>
Date:   Mon Feb 13 18:05:43 2017 +0000

    Cleanup: use the load exception instead of exposing ioexceptions/invalidformatexceptions

commit 847b552
Author: Tung Jin Chew <[email protected]>
Date:   Mon Feb 13 17:43:33 2017 +0000

    Fix typo; clear system colour instead of preset colour on the line chart template if specified. Didn't affect operation since the actual template doesn't use system colours; but could affect people using other templates in future.

commit b8fc101
Author: Tung Jin Chew <[email protected]>
Date:   Mon Feb 13 15:51:08 2017 +0000

    Mark anything that can be static as static, cleanup

commit 252682f
Author: Tung Jin Chew <[email protected]>
Date:   Mon Feb 13 15:34:26 2017 +0000

    Use InputStreamSource instead of Resource

commit b730cdb
Author: Tung Jin Chew <[email protected]>
Date:   Mon Feb 13 15:14:14 2017 +0000

    Split the powerpoint code into service implementation and UI controller wrappers; attempting to preserve history

commit ecf9b38
Author: Tung Jin Chew <[email protected]>
Date:   Mon Feb 13 13:41:16 2017 +0000

    Move the data transfer objects and slideshow template to their own package; in preparation for package extraction

commit f6deffb
Author: Tung Jin Chew <[email protected]>
Date:   Mon Feb 13 11:19:03 2017 +0000

    Fix the problem where only two of the three map views get rendered on the 3-map dashboard in Chrome. New fix works fine in Firefox and IE11as well.

commit 2ccdf36
Author: Tung Jin Chew <[email protected]>
Date:   Mon Feb 13 10:56:32 2017 +0000

    Note about the replace

commit 547731a
Author: Tung Jin Chew <[email protected]>
Date:   Fri Feb 10 20:15:07 2017 +0000

    Work through the node tree to try and get bold / italic / font size properties and linebreak correctly; all of which are lost if we use jQuery's .text() method. Reproduce the bold / italic / font size formatting on the exported PowerPoint.

commit d05d658
Author: Tung Jin Chew <[email protected]>
Date:   Fri Feb 10 18:10:07 2017 +0000

    The map operation is forced to be async, since thumbnailing the slides is an async operation. This means the Chrome popup blocker will block the new-window download to a _blank window. The downside is that an error will mean losing the current page.

commit f1f2125
Author: Tung Jin Chew <[email protected]>
Date:   Fri Feb 10 14:00:59 2017 +0000

    Enabling CORS on the map view seems to help with downloading multiple maps in parallel.

commit 6e7e7fc
Author: Tung Jin Chew <[email protected]>
Date:   Fri Feb 10 13:31:40 2017 +0000

    Allow text with zero margins; or margins without text. Allow customizing font-family and font-size.

commit 88bfc8e
Author: Tung Jin Chew <[email protected]>
Date:   Thu Feb 9 18:00:42 2017 +0000

    Tweak saved-search-link to correct position

commit 31e8bb5
Author: Tung Jin Chew <[email protected]>
Date:   Thu Feb 9 17:11:10 2017 +0000

    Make the title part of the SunburstData object, since it looks quite silly on the report to have multiple sunbursts without any title to tell them apart.

commit 8601d4b
Author: Tung Jin Chew <[email protected]>
Date:   Thu Feb 9 16:45:50 2017 +0000

    Misc. cleanup

commit 3602e9e
Author: Tung Jin Chew <[email protected]>
Date:   Thu Feb 9 16:26:12 2017 +0000

    Refactor dategraph so we can draw multiple dategraphs on a single slide, by working with raw XML objects. Allow reports to draw date graphs. Remove old loadTemplate() method.

commit 6a2a21d
Author: Tung Jin Chew <[email protected]>
Date:   Thu Feb 9 15:55:12 2017 +0000

    Ability to composite multiple sunburst charts onto a single report

commit eda0b34
Author: Tung Jin Chew <[email protected]>
Date:   Thu Feb 9 15:22:06 2017 +0000

    Rewrite the sunburst doughnut-chart handling code so it's done via processing raw XML objects; which should allow us to add multiple charts.

commit c114ecf
Author: Tung Jin Chew <[email protected]>
Date:   Thu Feb 9 11:25:08 2017 +0000

    Make it take 10%, or 70 pixels, whichever is bigger, unless that's more than 50% of the overall space, in which case limit it to 50%.

commit ab1a681
Author: Tung Jin Chew <[email protected]>
Date:   Thu Feb 9 11:11:20 2017 +0000

    Tweak column ratio

commit b748bcf
Author: Tung Jin Chew <[email protected]>
Date:   Wed Feb 8 21:03:28 2017 +0000

    Allocate more space to text and less to the count; for the common case.

commit 84c164b
Author: Tung Jin Chew <[email protected]>
Date:   Wed Feb 8 20:55:38 2017 +0000

    Crop the tables view to the anchor point heights in the table view; to the best that we can (which is pretty limited, since the POI table API doesn't account for word-wrap when estimating row heights).

commit c09d439
Author: Tung Jin Chew <[email protected]>
Date:   Wed Feb 8 19:47:44 2017 +0000

    Add anchor-rectangle bounds support for the list view; refactor it into a reusable method which can be run multiple times on a single slide. Only render its results and sort-by textboxes if they're not null; since the composite view disables it. Add list view rendering support for the report view. Sort the report's children so the chart-based views (sunburst, date graph) are rendered first; since any of the getShape() and createShape() methods run by the others will break things if the chart views subsequently add chart objects directly to the XML.

commit 33a296b
Author: Tung Jin Chew <[email protected]>
Date:   Wed Feb 8 17:05:39 2017 +0000

    Switch to an interface. Add 'report' api which allows placing multiple chart objects in different positions (varying levels of support for the different chart components at the moment).

commit a1b1e23
Author: Tung Jin Chew <[email protected]>
Date:   Wed Feb 8 16:16:29 2017 +0000

    Tag the various composable chart elements with an abstract class; give them a consistent naming convention.

commit 734180b
Author: Tung Jin Chew <[email protected]>
Date:   Wed Feb 8 16:08:33 2017 +0000

    Wrapper objects for topic map map and results view data, in preparation for composite slides

commit a000625
Author: Tung Jin Chew <[email protected]>
Date:   Wed Feb 8 11:20:19 2017 +0000

    Refactor table slide-drawing code into its own method with defined anchors and horizontal align

commit ff7f6a8
Author: Tung Jin Chew <[email protected]>
Date:   Tue Feb 7 20:17:33 2017 +0000

    i18n for table view. Use JSON request structure in preparation for multi-element slides.

commit 41637f0
Author: Tung Jin Chew <[email protected]>
Date:   Tue Feb 7 20:16:41 2017 +0000

    Cleanup: header year

commit 8db3e4a
Author: Tung Jin Chew <[email protected]>
Date:   Tue Feb 7 19:57:21 2017 +0000

    Cleanup: dedupe internal workbook rewrite

commit f728910
Author: Tung Jin Chew <[email protected]>
Date:   Tue Feb 7 19:45:44 2017 +0000

    Fix marker offset calculations if the object isn't full-width

commit ce24968
Author: Tung Jin Chew <[email protected]>
Date:   Tue Feb 7 19:31:50 2017 +0000

    Refactor map slide-drawing code into its own method with defined anchors and horizontal align-center vertical align-top; in preparation for multi-object charts

commit de9997c
Author: Tung Jin Chew <[email protected]>
Date:   Tue Feb 7 18:44:58 2017 +0000

    Refactor topicmap slide-drawing code into its own method with defined anchors; in preparation for multi-object charts

commit 6a68c50
Author: Tung Jin Chew <[email protected]>
Date:   Tue Feb 7 18:03:50 2017 +0000

    i18n for sunburst pptx export; change data format to take a JSON object in preparation for multi-slide write

commit 85f6594
Author: Tung Jin Chew <[email protected]>
Date:   Tue Feb 7 16:34:26 2017 +0000

    Use the same base presentation for all powerpoint exports; primarily so we'll inherit the page size settings, but will also give us any other random bits of the slide template.

commit 5d1375a
Author: Tung Jin Chew <[email protected]>
Date:   Mon Feb 6 20:34:32 2017 +0000

    Fix encoding issues which were causing e.g. corrupt Unicode characters in the list view and nonsensical rendering of non-breaking-space characters in the date graph; turns out the CharacterEncodingFilter wasn't being called in time on multipart/form-data requests .

commit c65e379
Author: Tung Jin Chew <[email protected]>
Date:   Mon Feb 6 18:17:02 2017 +0000

    Use a single PowerPoint template containing the graphs instead of multiple templates.

commit 880d34c
Author: Tung Jin Chew <[email protected]>
Date:   Mon Feb 6 11:33:02 2017 +0000

    Open map view in _self, otherwise the Chrome popup blocker will complain (otherwise we'd use _blank so we don't lose our current page in case an error happens).

    (cherry picked from commit 675b9e8)

commit 2763c94
Author: Tung Jin Chew <[email protected]>
Date:   Mon Feb 6 11:14:47 2017 +0000

    Correct paths so PowerPoint export works when reverse proxied.

    (cherry picked from commit 7c8b10c)

commit c783374
Author: Tung Jin Chew <[email protected]>
Date:   Mon Feb 6 12:03:52 2017 +0000

    Fix path to the saved-search-status page when using a reverse proxy.

    (cherry picked from commit 44fd04e)

commit f68d21e
Author: Tung Jin Chew <[email protected]>
Date:   Fri Feb 3 21:19:19 2017 +0000

    There's encoding issues; the non-breaking-spaces are being parsed incorrectly server side since it's not recognizing the UTF-8 encoding. Workaround for now is to use normal spaces in the PowerPoint.

commit cfcfd9c
Author: Tung Jin Chew <[email protected]>
Date:   Thu Feb 2 16:32:57 2017 +0000

    Catch runtime exception rather than exception; and just suppress it rather than logging

commit ac23b46
Author: Tung Jin Chew <[email protected]>
Date:   Thu Feb 2 16:01:20 2017 +0000

    Add highlighting support for the bold text elements tagged with <HavenSearch-QueryText-Placeholder>. Coalesce newlines and multiple whitespace in the summary to a single whitespace to match HTML rendering behaviour. Slight refactoring to make things neater.

commit 2bb0cd5
Author: Tung Jin Chew <[email protected]>
Date:   Thu Feb 2 15:04:48 2017 +0000

    Use relative dates for consistency with the exported UI (admittedly they make less sense if the PowerPoint is viewed later; but it's closer to the normal display)

commit e6732f3
Author: Tung Jin Chew <[email protected]>
Date:   Thu Feb 2 14:56:25 2017 +0000

    Use page dimensions from the slides rather than hardcoded values

commit 2412d8c
Author: Tung Jin Chew <[email protected]>
Date:   Thu Feb 2 14:43:34 2017 +0000

    Use paragraphs for different text so we can conditionally import them and apply margins; tweak thumbnail scale factor.

commit 9078125
Author: Tung Jin Chew <[email protected]>
Date:   Thu Feb 2 14:13:27 2017 +0000

    Add thumbnail support to list PowerPoint export

commit dc041d8
Author: Tung Jin Chew <[email protected]>
Date:   Thu Feb 2 11:57:47 2017 +0000

    We don't need to reserve the margin; we just need enough space for the document.

commit b19fec6
Author: Tung Jin Chew <[email protected]>
Date:   Thu Feb 2 11:32:23 2017 +0000

    Ensure the parametric graphing buttons only appear if you have the graph view enabled

commit 750692f
Author: Tung Jin Chew <[email protected]>
Date:   Thu Feb 2 11:01:41 2017 +0000

    If the user doesn't have the BI role, hide the powerpoint button, since all export stuff is restricted to BI users.

commit baad263
Author: Tung Jin Chew <[email protected]>
Date:   Wed Feb 1 19:54:08 2017 +0000

    Working multi-page list PowerPoint export !

commit ca57e7a
Author: Tung Jin Chew <[email protected]>
Date:   Wed Feb 1 19:01:31 2017 +0000

    Cleanup copyright

commit 23155e1
Author: Tung Jin Chew <[email protected]>
Date:   Wed Feb 1 13:41:27 2017 +0000

    The topic map changes have been made into a pull request upstream which Alex has merged; so we can just update to that version and delete the local variants.

commit 9b43b72
Author: Tung Jin Chew <[email protected]>
Date:   Wed Feb 1 12:10:55 2017 +0000

    Apply changes suggested by Alex

commit 99a6a3d
Author: Tung Jin Chew <[email protected]>
Date:   Wed Feb 1 11:47:33 2017 +0000

    Update to the new code which is configurable; which has been made in to a pull request

commit 1c4e02e
Author: Tung Jin Chew <[email protected]>
Date:   Tue Jan 31 14:05:26 2017 +0000

    Actually, we have to find the marker z-index and sort the markers by it; otherwise the markers on the compare-query 'exclusive to saved search / common to both' may not appear in the correct order.

commit 84f0ece
Author: Tung Jin Chew <[email protected]>
Date:   Tue Jan 31 12:08:51 2017 +0000

    The cluster markers should be pushed to the front of the marker list so that they appear behind the normal markers (they potentially overlap if a cluster marker has been expanded)

commit f5919eb
Author: Tung Jin Chew <[email protected]>
Date:   Tue Jan 31 11:53:16 2017 +0000

    When you click on a cluster marker, it expands its children but also fades out in the UI. Replicate the faded-out state in PowerPoint

commit 81ef1c7
Author: Tung Jin Chew <[email protected]>
Date:   Mon Jan 30 18:16:22 2017 +0000

    Further tweak topicmap for speed

commit 28553e4
Author: Tung Jin Chew <[email protected]>
Date:   Mon Jan 30 14:58:13 2017 +0000

    Restrict to only supporting POST on the export methods

commit 4c4e758
Author: Tung Jin Chew <[email protected]>
Date:   Mon Jan 30 14:55:07 2017 +0000

    It's more efficient to use form-data for these large POST uploads

commit 6007732
Author: Tung Jin Chew <[email protected]>
Date:   Mon Jan 30 14:02:50 2017 +0000

    Increase limits to 16MB, since experimenting with random data to create a 1920x1080 PNG gives a 12MB file; so 10MB might not be enough and 16MB seems a sensible upper limit.

commit 00eb7ed
Author: Tung Jin Chew <[email protected]>
Date:   Mon Jan 30 12:45:04 2017 +0000

    In addition to the Spring form parsing side, we also have to increase the embedded Tomcat connector's max post size (otherwise it defaults to 2MB, which some map exports can exceed; e.g. when testing in IE11 on a 1600x900 screen) .

commit 750963e
Author: Tung Jin Chew <[email protected]>
Date:   Mon Jan 30 11:56:03 2017 +0000

    A lossless PNG actually compresses to be smaller than a high quality 1.0 JPEG of similar resolution, which explains why the source map tiles were served as PNG as well. Seems to work in Chrome, Firefox and IE11

commit 481550f
Author: Tung Jin Chew <[email protected]>
Date:   Mon Jan 30 11:42:44 2017 +0000

    It turns out the reason the images were being truncated when we used image/jpeg with quality 1.0 is because there's a default 524288 length limit for a input field; and there's a bug in Chrome (https://bugs.webkit.org/show_bug.cgi?id=44883) which prevents us increasing this limit. Switch to a textarea instead to work around the limit. We also have to increase the Spring Boot max file input length limit from 1MB since on a high-res screen a high-quality JPEG may take ~2MB; set it at 10MB arbitrarily.

commit 691fe00
Author: Tung Jin Chew <[email protected]>
Date:   Mon Jan 30 10:40:07 2017 +0000

    Excessively high quality settings seem to cause problems with the image; only part of it appears.

commit a1214e2
Author: Tung Jin Chew <[email protected]>
Date:   Fri Jan 27 19:04:59 2017 +0000

    Ask for maximum quality JPEG export (which sadly is still not lossless). We can't export PNG lossless since it contains an alpha layer, and OpenOffice doesn't seem to handle alpha channels

commit 139f725
Author: Tung Jin Chew <[email protected]>
Date:   Fri Jan 27 18:23:46 2017 +0000

    Make font colour customizable; since it's black on the normal map but white on the compare-results map

commit 90196a5
Author: Tung Jin Chew <[email protected]>
Date:   Fri Jan 27 17:56:12 2017 +0000

    Use grouped shapes to give the mix of opaque-text-center and semitransparent-larger-circle to match the UI. Set font to be white, since the compare map uses white font.

commit aa88e5d
Author: Tung Jin Chew <[email protected]>
Date:   Fri Jan 27 17:24:48 2017 +0000

    Use appropriate colours for the markers when exporting in comparison map view (there's no way to get the data programatically for the single-point markers, since it's a background image not the literal CSS colour mapping; so we just use a table of CSS class names to image background colours)

commit a9a2825
Author: Tung Jin Chew <[email protected]>
Date:   Fri Jan 27 17:22:02 2017 +0000

    Center the title text in PowerPoint

commit 6992fbd
Author: Tung Jin Chew <[email protected]>
Date:   Fri Jan 27 16:19:24 2017 +0000

    Add powerpoint export to comparison view

commit 167b26e
Author: Tung Jin Chew <[email protected]>
Date:   Fri Jan 27 16:18:58 2017 +0000

    Fix title, use textarea to allow multiline input.

commit ea05709
Author: Tung Jin Chew <[email protected]>
Date:   Fri Jan 27 15:49:01 2017 +0000

    Disable console logging

commit 90d1308
Author: Tung Jin Chew <[email protected]>
Date:   Fri Jan 27 15:45:48 2017 +0000

    Move marker code to the map-view.js; which is a better place for it now that we've done away with the requirement for an external marker list

commit e7d8c17
Author: Tung Jin Chew <[email protected]>
Date:   Fri Jan 27 15:30:28 2017 +0000

    Refactor to use the map layers directly instead of using references to external markers

commit 823a46b
Author: Tung Jin Chew <[email protected]>
Date:   Fri Jan 27 15:14:09 2017 +0000

    Tweak selector logic in preparation for refactoring into common area

commit 58f5559
Author: Tung Jin Chew <[email protected]>
Date:   Fri Jan 27 12:37:16 2017 +0000

    Tweak the markers so the cluster markers are centered on the target lat-long, whereas the single-point markers have the tail of the marker pointing at the target lat-long. Add semitransparency to the cluster markers matching the original; keeping a lighter semitransparency effect on the single-point marker fill since it looks nice.

commit ab68761
Author: Tung Jin Chew <[email protected]>
Date:   Fri Jan 27 12:10:15 2017 +0000

    Rotate point markers so they point downwards; give them a dark line color edge, add alpha shading, make their anchor size slightly smaller than the round circle types (which makes them about the same size in the PowerPoint).

commit 47c8b06
Author: Tung Jin Chew <[email protected]>
Date:   Fri Jan 27 12:01:25 2017 +0000

    Tweak tolerances so markers are only drawn when outside the bounds if they're within 0.1% of the border. Update comments.

commit ffbef8e
Author: Tung Jin Chew <[email protected]>
Date:   Fri Jan 27 11:16:31 2017 +0000

    Workaround htmlcanvas not working with 3d transforms on Firefox and IE11 by disabling the use of 3d transforms in those browsers; required so that we can correctly export the background image to PowerPoint on those browsers (otherwise it'll have blank areas and/or show the wrong area of the screen).

commit a0a3b65
Author: Tung Jin Chew <[email protected]>
Date:   Thu Jan 26 20:06:49 2017 +0000

    Create a hyperlink linking back to the slide; which really is just to get us a hover tooltip on the leaf marker so we can put the title text of the document on it.

commit 24ea2dd
Author: Tung Jin Chew <[email protected]>
Date:   Thu Jan 26 20:01:54 2017 +0000

    Use the map API to compute lat/lon positions directly rather than trying to compute backwards from rendered markers; gives proper locations cross-browser.

commit bb6295d
Author: Tung Jin Chew <[email protected]>
Date:   Thu Jan 26 18:20:41 2017 +0000

    Improve marker rendering

commit 79cdfb0
Author: Tung Jin Chew <[email protected]>
Date:   Thu Jan 26 17:31:16 2017 +0000

    Marker export; has some offset/scaling issues due to leaflet's use of translate3d / translate CSS transforms which jquery's .position doesn't seem to account for

commit fa45215
Author: Tung Jin Chew <[email protected]>
Date:   Thu Jan 26 12:48:45 2017 +0000

    Re-add the tile layer local proxy

commit 1e0d4bd
Author: Tung Jin Chew <[email protected]>
Date:   Thu Jan 26 12:40:57 2017 +0000

    Partial workaround for rendering issues on IE11; Chrome is fine

commit 7aa06f1
Author: Tung Jin Chew <[email protected]>
Date:   Thu Jan 26 11:53:33 2017 +0000

    Working map tile export

commit 366560a
Author: Tung Jin Chew <[email protected]>
Date:   Wed Jan 25 19:49:04 2017 +0000

    html2canvas-compatible base64 image proxy; works to get around CORS image tile fetch problems

commit d72afd6
Author: Tung Jin Chew <[email protected]>
Date:   Wed Jan 25 19:44:15 2017 +0000

    Tweak topicmap timings for faster rendering

commit dbabc5a
Author: Tung Jin Chew <[email protected]>
Date:   Wed Jan 25 18:50:27 2017 +0000

    Implementation of the Express-style proxy ala https://github.com/niklasvh/html2canvas-proxy-nodejs/blob/master/server.js , but it seems to return an object; and the html2canvas code expects a string

commit 7961352
Author: Tung Jin Chew <[email protected]>
Date:   Wed Jan 25 18:21:57 2017 +0000

    Add html2canvas library. To get around the CORS problem, we proxy the images locally.

commit 0bb13d1
Author: Tung Jin Chew <[email protected]>
Date:   Tue Jan 24 19:49:17 2017 +0000

    Multicolumn table export

commit 054c49a
Author: Tung Jin Chew <[email protected]>
Date:   Tue Jan 24 19:15:29 2017 +0000

    Improved rendering. Button to export tables.

commit 98503e6
Author: Tung Jin Chew <[email protected]>
Date:   Tue Jan 24 18:13:03 2017 +0000

    First draft of Powerpoint table export

commit 5602acd
Author: Tung Jin Chew <[email protected]>
Date:   Tue Jan 24 17:24:48 2017 +0000

    Tweak wording; add sanity check

commit bafa238
Author: Tung Jin Chew <[email protected]>
Date:   Tue Jan 24 13:42:16 2017 +0000

    Fix the embedded spreadsheet info and ranges; previously pressing 'Edit Data' in PowerPoint caused the chart to be messed up

commit 6fbfbdc
Author: Tung Jin Chew <[email protected]>
Date:   Tue Jan 24 13:16:22 2017 +0000

    One-dimension sunburst export

commit dea09c2
Author: Tung Jin Chew <[email protected]>
Date:   Tue Jan 24 12:04:41 2017 +0000

    Fix corruption of the template powerpoint; it was due to resource filtering, so we need to configure the maven-resources-plugin not to filter pptx files. Move it back to the templates folder, and use it in the sunburst export step.

commit 0e0162e
Author: Tung Jin Chew <[email protected]>
Date:   Tue Jan 24 11:13:47 2017 +0000

    First draft of sunburst single-level view export; requires a template file (which currently has to be loaded from the filesystem due to issues with resource fetching returning the wrong number of bytes).

commit 196341c
Author: Tung Jin Chew <[email protected]>
Date:   Fri Jan 20 19:38:45 2017 +0000

    Match the gradient angle from my original topicmap

commit 30090dd
Author: Tung Jin Chew <[email protected]>
Date:   Fri Jan 20 19:32:40 2017 +0000

    Gradients with semitransparency!

commit f01b6da
Author: Tung Jin Chew <[email protected]>
Date:   Fri Jan 20 19:17:44 2017 +0000

    White font with bold text; requires using the objects directly

commit f55bc0a
Author: Tung Jin Chew <[email protected]>
Date:   Fri Jan 20 18:50:38 2017 +0000

    GET has its own issues; it doesn't work for complicated topicmaps in IE (tested with IE11) since there's a URL GET max length limit. Switch back to POST; we can make the POST request work by appending the temporary form to the page temporarily (if it wasn't appended; it'd work in Chrome but not Firefox or IE11). Now works in all three browsers.

commit 4e35423
Author: Tung Jin Chew <[email protected]>
Date:   Fri Jan 20 18:13:26 2017 +0000

    Cleanup

commit 2497389
Author: Tung Jin Chew <[email protected]>
Date:   Fri Jan 20 18:12:48 2017 +0000

    Firefox doesn't allow the injected <form method="post" target="_blank"> to open a new window; for some reason (presumably security) so we have to use GET instead

commit 3066f10
Author: Tung Jin Chew <[email protected]>
Date:   Fri Jan 20 18:04:29 2017 +0000

    Show inline if possible

commit f98eb64
Author: Tung Jin Chew <[email protected]>
Date:   Fri Jan 20 18:04:17 2017 +0000

    Use the modified copy of the topicmap

commit 2e56a85
Author: Tung Jin Chew <[email protected]>
Date:   Fri Jan 20 17:45:45 2017 +0000

    Commit copy of published topicmap

commit 452241f
Author: Tung Jin Chew <[email protected]>
Date:   Fri Jan 20 17:41:31 2017 +0000

    Topicmap rendering as PPTX
…tead of relying on the correct font size offsets.
…en the input changes incl. when the up/down buttons are used, rather than just on focus loss
…data is available; map is left enabled in case the user wants to export a blank map.
…he margins taking all of the page. Tests for the margin validation and template validation logic.
…bottom margins. Add tests for the PowerPointConfigValidator.
…t saves us having to pass every tile through the webserver; only uses it on the ones we want to render into PowerPoint). Add more tests.
…ate slide in a presentation instead of drawing everything into a single slide.
…/form for SlimerJS to be able to get the post body; otherwise the file upload data is prematurely truncated in the request info.
… for various characters like '/', '\', '=' etc.) than application/x-www-form-urlencoded . Since SlimerJS can't parse this correctly, we have to store the request data somewhere on the page where SlimerJS can access it, in this case on the report export button group.
…er to export their data into reports once they have widgets created for them.
…'s changes, after rebasing in the 'feature/parametric-performance' changes
…ay with the store-request-data-on-report-group workaround.
…omise before showing the report export button, which waits for the widget's data to load (e.g. so the timing is correct to export a sunburst dashboard widget)
…list-widget correctly waits for the result list data to be available; makes export of the results-list-widget via CasperJS more reliable.
…erpoint export so they're visible in the scheduled exported report.
…ed transitions to allow hiding the non-leaf nodes when clicked in PowerPoint.
@tung-jin-chew-hp
Copy link
Contributor Author

Ivo's merged in the dashboard export as 3ef79fb ; though this doesn't include the per-view export (which is fair enough) nor the powerpoint custom template-file/margins widget in the settings (which I think should be added).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant