-
Notifications
You must be signed in to change notification settings - Fork 0
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
v2 discussion #8
Comments
@starryeyez024 thanks for you comments, that sounds really good we will look into them We definitely need to look into the design, we didn't do any work on that yet. If you're interested, we could definitely use some help there. |
you mean the npm downloads per week? sounds like a good idea 👍
sadly we see that developer in a lot of cases do not take size into consideration... apparently, size in kb is not meaningful enough? Pika also uses in a very visual way (the bars on the right) https://www.pika.dev/search?q=test still it seems not "imporant" enough so we wanted to try something different - the idea is actually from bundlephobia https://bundlephobia.com/[email protected] but yeah I agree that it's 2 times the same info and it takes quite some space :/ So if there is any other idea of how to bring more awareness to size we are all yours 🤗 the importants for this is quite well demonstrated by Alex Russell https://youtu.be/wsdPeC86OH0
we check for there is some more info if you scoll down to "Demo" under
very good idea 💪 also probably add an extra banner for this special case (e.g. similar to [lib|lit-element] there could be [lib|0 dependencies] or so)
we probably could replace the download time with the npm downloads per week? for putting results in boxes on mobile... it probably reduces the available space too much so I think keeping a line or different colour between them should be fine it's quite inspired by https://www.checkfelix.com the way filters are used there is quite nice imho :) |
so I played around with adding boxes for mobile and you are absolutely right it's nicer 🤗 => also added full-screen detail view 💪 and in this view, you can swipe left/right to go the next/previous result 🎉 for what to display as the main metrics I'm still unsure 😅
so it would be
|
Is size going to be total size including dependencies, is that even possible to grab? |
yes the size will be including dependencies :) info is courtesy of bundlephobia which as far as I know does npm install and bundles main/module entry point to get the real size. |
Size! Example: Which router to use? Better featured router comes in at 8k, but I can squeeze by with 1k, I think maybe? |
quite a big update for mobile 💪
|
Wow you guys move fast! A response to one of your questions, if its not too late:
Thanks again, this is great! |
thhxxx 🤗
yeah I think we will need to tackle it when we will think about the actual design (which we will start soonish) - e.g. we will need to make sure it visually gets the attention it deserves 💪 Updates:
Without further input - this will be considered the end of the technical POC and we will swich focus to design soonish. PS: added a nice gif to webcomponents/webcomponents.org#1250 (comment) |
We have been working some time now to improve on our catalog and make it "real"... still a POC for gathering info but feedback is much appreciated.
This shows the overall idea
custom-elements.json
) and then you can search./add.html
.demo/index.html
or any url specified in package.json as (demoUrl)=> styling is basically non-existing
See it in action
https://catalog.open-wc.org/
See the items docs
https://catalog.open-wc.org/storybook/?path=/docs/item--single
and all the other parts
https://catalog.open-wc.org/storybook
For now please no design feedback - we are still trying to find out which information we should gather and display. Also, which filters are most useful.
So if you could publish and index a test package, or a fork or your real component and provide feedback that would be super nice.
Also what do you think of the mobile behavior?
The text was updated successfully, but these errors were encountered: