-
Notifications
You must be signed in to change notification settings - Fork 201
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
Flexible Card Request #249
Comments
Yes, yes! This will be very much a great help for me. There is any plan for moving forward with this feature? |
I would also like to see a way to change card icons and maybe some kind of subtabs or a better way to divide, organize, or clearly separate areas within a tab |
Me too! |
Pro user here as well, and I understand if these features need to remain Pro-only. Currently, I'm using checkboxes to show/hide extra cards, and employing tabs to the fullest of their potential, but my project is still looking a bit untidy. Any one of these options could kick it up to a high degree of polish. I'm using this in a substantial (for me) commercial product, and it really is a fantastic tool -- I really appreciate all the work Mathieu and Ayush have put into it. I have plenty of experience with HTML and CSS, but little to none with Svelte and even JS broadly. So large parts of the source are outside my areas of expertise. I would expect that any of these new card types would require adding to the compressed page HTML, and to the Svelte files (in addition to the obvious cpp class constructors and methods). I would be willing to contribute, but would need some guidance on the JS side, unfortunately. |
Mathieu, regarding your idea of a divider card with headings: I think this could work well incorporating my suggestion #2. The way I have envisioned this working (with two card indices as arguments) would be to simply add the opening div before the first indexed card, and the closing /div after the last indexed card. That way, the background/border style can simply wrap all of the cards in between the two indices (inclusive of the first and last cards). With headings, that would allow very simple (as far as the developer is concerned) grouping of other cards under a single heading/subheading. That makes for a very intuitive/tidy under experience, where the end-user comprehends items in relation to each other, according to the Law of Common Region. |
Very nice! Would you be willing to send me the files you modified, or walk me through your changes? |
I would have walked you through the changes (cannot send the files as per the license), but the problem is that this work is based on ESP-DASH Pro v5 codebase (ref: #246) which is not released yet (only @ayushsharma82 has access to and can release - I've just shared all these improvements with him). The way to write components is different in v5. |
Hi @finchamp & @mathieucarbou , I appreciate the feature requests but please note that ESP-DASH Pro is not an open-source project. All implementations must go through the review process & testing before final release. I’ll suggest anyone interested to wait for the official release where I implement features & bug fixes proposed by mathieu, along with revamped UI. |
Ah, that's very helpful information. Well, then, I'm looking forward to v5! Again, my experience is primarily in C++14/17, so if I can contribute, let me know. If you need proof of my licensure, I can provide that privately.
Thanks, I fully understand. I have no intention of publishing files from my Pro installation, even modified. I understand that this would likely be better discussed on the Pro/v5 branch, but I understand that that access is privileged. If I can be of assistance, let me know. Otherwise, I will watch for the release of v5 and go from there. |
But you really don´t need V5 for a separator. I´ve managed to use a generic card as a separator. Only downside is, that I would like to change or disable icons. A separator card without an icon or a simple "-" would be nice. ![]() PS: @mathieucarbou Why is your overview tab icon a house? |
Thanks for sharing your approach. I've done something similar, using buttons to show/hide a number of related cards. Due to the size of my project, it's just too visually cluttered to use simple status/separator cards, which is why I didn't request them in my original post. My interest in Mathieu's modifications was primarily to better understand the workflow of the tool, and how I might attempt to implement some slightly more robust grouping/style to my project. Since that is changing with v5, I am not pursuing it any further for now. Although I wouldn't mind knowing how to remove the icon from a status card without using CSS hacks. |
Good question! I never customised it... Oh wait! I am running the latest pro version: is it possible that you did not download the latest zip file ? |
This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days. |
Hello guys,
A few ideas for some features that would be really helpful to me:
If you want to point me in the right direction, I could see if this was within my ability to provide some code for review. Thanks!
The text was updated successfully, but these errors were encountered: