Skip to content

Used polymer modulizer to convert the element to Polymer 3#8

Closed
DrNiels wants to merge 10 commits into
mlisook:masterfrom
DrNiels:master
Closed

Used polymer modulizer to convert the element to Polymer 3#8
DrNiels wants to merge 10 commits into
mlisook:masterfrom
DrNiels:master

Conversation

@DrNiels

@DrNiels DrNiels commented Jun 1, 2018

Copy link
Copy Markdown

I converted the element to Polymer 3, using the polymer modulizer. Fortunately, it went through without any problems.

This PR includes #7 as I used the modulizer on my fork.

You would need to change the name of the npm package to use it.

Otherwise, it was possible that percentual heights were not propagated correctly as the saw-div did not have the same height as the resize-aware element.
Also fixed the CSS style which refered to the element saw but was probably meant for the div with the id "saw"
@mlisook

mlisook commented Jun 2, 2018

Copy link
Copy Markdown
Owner

Thank you for this PR!

I expect to complete testing, merging and publishing this on 6/2.

@mlisook

mlisook commented Jun 2, 2018

Copy link
Copy Markdown
Owner

Looking good, though the demo is not running ...

@DrNiels

DrNiels commented Jun 4, 2018

Copy link
Copy Markdown
Author

Some import paths were broken while modulizing, I guess this applies to the scripts in the demo as well. I will look into it, when I find some time.

@mlisook

mlisook commented Jun 8, 2018

Copy link
Copy Markdown
Owner

I've decided to leave this element at Polymer 2.0 and release plastic-resize-aware for Polymer 3.0, which is needed for the next version of plastic-map-info and a work project in progress.

plastic-resize-aware uses a ResizeObserver in lieu of scroll detection and includes an element queries function as plastic-element-query.

@DrNiels

DrNiels commented Jul 8, 2020

Copy link
Copy Markdown
Author

As you already released your Polymer 3 version quite some time ago, I'll just close here. Thanks for your work!

@DrNiels DrNiels closed this Jul 8, 2020
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.

3 participants