uprtcl / js-uprtcl

Libraries, tools and modules to create _Prtcl web apps
http://www.uprtcl.io/
Other
43 stars 13 forks source link

Image and Video URL #149

Closed wtubog closed 4 years ago

wtubog commented 4 years ago

For #105 and #106

pepoospina commented 4 years ago

Hi @wtubog, it's looking really good.

I suggest we simplify the size configuration and offer two fixed heights: 200px and 600px. with a small and large icons. Large by default maybe.

The width would be given by the photo or would be limited by the editor width, leaving the resulting height even if smaller than 200px or 600px.

The height to width ratio should not change.

Here is an example of what I have in mind https://docs.google.com/drawings/d/1a3ScwVSUYt5QYotDymBCZFO5QQsHRwTvm6dRu2USr4c/edit

I think this is possible to be done using CSS only and not JS.

Also, maybe we can use material icons for small and large sizes large https://material.io/resources/icons/?search=size&icon=crop_3_2&style=baseline small https://material.io/resources/icons/?search=size&icon=crop_7_5&style=baseline

Let me know what you think. I might be proposing something inconsistent here.