polymer-quill icon indicating copy to clipboard operation
polymer-quill copied to clipboard

This is a very good component. Do you have any plans to migrate to Polymer 2.0-rc?

Open phani1kumar opened this issue 7 years ago • 12 comments

Hi,

I tried to change the bower dependencies to 2.0-rc2 and tested, but the component is not working as expected. Can you please confirm if there are any plans to upgrade this to 2.0-rc?

Thanks and Best Regards, Phani

phani1kumar avatar Mar 16 '17 12:03 phani1kumar

I plan to upgrade to Polymer 2.0, but will not have a chance to work on it until April. Hopefully by then the 1.x to 2.0 conversion utility will be available .

chuckh avatar Mar 16 '17 12:03 chuckh

Thank you very much for such a quick response. I was able to make the html-render example work. But rest of the demos are not working.

I am currently developing a product for which the polymer-quill-bubble-firebase and polymer-quill-html-render seems to be a perfect fit, I'll try to build a small miniature component for this functionality alone. Hope the conversion utility will be coming soon :)

phani1kumar avatar Mar 16 '17 13:03 phani1kumar

+1 Interested if there's a way to help the port to 2.0

eob avatar Mar 21 '17 00:03 eob

+1 my js skills are too lame to offer much help but would be delighted to help test when it is time.

There are notes in the 2.0 docs about shady dom - which my implementation of polymer-quill uses, but not sure how to interpret.

petecarapetyan avatar Mar 22 '17 09:03 petecarapetyan

Yes, please make sure to tell us if you need help with something specific :) very interested in this too

yorrd avatar May 10 '17 17:05 yorrd

Sorry I have not been able to do this. Help would be great, @yorrd and @petecarapetyan you could fork polymer-quill and try running polymer lint --fix and see what it does. If it works create a pull request, if not report back on the problems. See more below about polymer lint --fix.

From @rictic

In the process of developing the upgrader we realized that it had a ton of overlap with the new linter (registry, configuration, etc). So the plan is to be able to run polymer lint to discover any issues your code has with your chosen version of Polymer (1.0, 2.0, 2.0-hybrid), and to be able to run polymer lint --fix to automatically fix as many of those issues as have unambiguous fixes.

You can try out the new linter by running polymer experimental-lint in polymer-cli@next, and in the next couple of weeks it will replace the old linter, and run as polymer lint.

see this discussion at: https://github.com/Polymer/polymer-upgrade/issues/4#issuecomment-285787035

chuckh avatar May 10 '17 18:05 chuckh

polymer 2 port for component available at https://github.com/DannyMcwaves/polymer-quill

DannyMcwaves avatar Nov 21 '17 16:11 DannyMcwaves

@DannyMcwaves going to test it if you interested

giona69 avatar Dec 30 '17 15:12 giona69

@giona69 It did not work at all for me. Same error "Uncaught SyntaxError: Unexpected identifier".

Hammond2789 avatar Jan 09 '18 15:01 Hammond2789

@Hammond2789 no it didn't work. What I did, it is to get https://github.com/quilljs/quill build it locally, and then replace the css and js in polymer-quill.html ... it works but you have to get the fork they are working on for the shadowRoot problem ... it works almost everywhere, except safari, see https://github.com/quilljs/quill/pull/1805

giona69 avatar Jan 09 '18 15:01 giona69

I have created branch for pure Polymer 2.x at https://github.com/chuckh/polymer-quill/tree/polymer-2.x. Use this to do Polymer 2.x work for polymer-quill. This does NOT work yet.

@Hammond2789 could you try this branch with quill fork https://github.com/quilljs/quill/pull/1805. See if it works for you. I have enabled ShadyDOM so it should work for Safari too.

Status

[] Use ShadowDOM Quill fork and test [] Change to ShadowDOM Quill fork if it works [] Get Bubble popup working [] Get Snow toolbar working [x] Demos displaying Snow toolbar [x] Demos display content correctly [x] Create Polymer 1.x branch for Polymer 1.x version [x] Test with Quill 1.3.4 (DOES NOT WORK) [x] Test with @chrismbeckett's fork (https://github.com/quilljs/quill/compare/develop...arsnebula:develop) - DOES NOT WORK [x] Update Readme [x] Change dependencies to Polymer 2.x and Polymer 2.x elements [x] Update Demo files for Polymer 2.x syntax [x] Add <custom-style></custom-style> where needed [x] Added <dom-bind></dom-bind> where needed [x] Change @apply(--) to @apply -- [x] Remove :host ::content prefix from styles files [x] Add <link rel="import" href="../polymer/lib/utils/async.html"> import where needed [x] Change to for ShadyDOM for Polymer 2.x [x] Update sample.txt content to load better [x] Change HMTL Render to work with inner-h-t-m-l instead of <marked-element> [x] Remove

see all changes at https://github.com/chuckh/polymer-quill/compare/polymer-2.x?expand=1

chuckh avatar Jan 20 '18 20:01 chuckh

Just now checking in with this thread. From reading the previous messages still not sure if I should try out one of the Polymer2 versions yet? If anything is ready to test out in next couple weeks I should be available, if it would be helpful. I looked at jumping in at the code level but my javascript skills are currently way sub-par for the level of work required. :(

petecarapetyan avatar Jan 21 '18 02:01 petecarapetyan