-
Notifications
You must be signed in to change notification settings - Fork 5
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
[RFC] Package list enhancements #59
Comments
This should be a lot easier to do with the work completed recently, and also @rossriley is going to have a look at the search functions soon too. @nbehier should we be able to agree (I stay out of UI, but will get buy-in from the others), are you interested in tackling some of the UI work? ping @PhillippOhlandt |
If I can help, I'll be happy to contribute! |
Hmm, I wouldn't start replacing single components in the store yet. Otherwise, every page will look differently. We should hold stuff like that back to a major redesign and restructuring of the store. |
Connecting to #56 then |
@PhillippOhlandt Yes, but we can try and redo everything, and never get it done, or we could be a little strategic and get bits started now. I agree that it should stay consistent though. |
@PhillippOhlandt I'm totally agree with you, list page and homepage redesign have to serve the same goal => store redesign. Do you want to sum up all ideas on #56 or a new global issue ? |
@GawainLynch the new store homepage structure would still have the same (ugly :D) style. The proposed listview above can't be done with semantic UI in that form. There are also some things like "downloads" and "starred" that will fuck performance at the current state. |
@nbehier when you show me a "non-mockup" without downloads and starred counts, we can see if it can be already added to the store yet. |
@nbehier please dont get me wrong. i like the idea of improved listviews because the current ones arent optimal at all. when we want to improve them right now, we need to find something in the middle of "current" and "perfect" that works for the current theme. |
Yeah, but lets iterate rather than try to land perfect now … too much to do and not enough time, and if people like @nbehier are willing to help with bits then I say let's at lest get it moving in a branch |
@GawainLynch i agree, read my last two replies :) |
I was typing while you were too 😆 |
Could be doable but please with a list view and not a grid view for extensions. Could you please look if such a layout is possible with SemanticUI (which we use at the current store)? |
I think we could use :
|
please no cards for the listview, otherwise it wouldn't be a list anymore. |
OK, so we can use list with image as current page... |
thats not the type of list we need. The current list view isn't that bad. it just needs the sidebar and some new values. Why not cards: because 99% of all extensions don't have an image and we would need the cards in a grid, which is suboptimal here. |
@PhillippOhlandt I think with the latest extensions more then 50% has an image. On the other hand this could be a nice way to push submitters a bit more to use images. Beside that I think the cards or item(2 columns) of semantic UI are a nicer way to show the extensions then a list, especially on a larger screen. |
I am not sure if we should change something now. The store needs a redesign and it is very likely that we don't stick with semantic ui. |
Fair enough, but is there a timeline, or anything more specific? |
Not really. I am mostly waiting for the styleguide and the Bolt backend CSS refactor. |
Every list page could be enhanced. We could use a JS script to easily filter/search extensions/themes list. Mockup attached to launch discussion.
The text was updated successfully, but these errors were encountered: