-
-
Notifications
You must be signed in to change notification settings - Fork 214
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
New state for modules #10934
Comments
@pascalgrimaud : I believe it is an excellent idea! I have often felt the need to release something with beta status at places where I've worked, but it was never allowed because they preferred to keep testing internally until the functionality was complete.
I think the term "beta" encourages testing more than "wip" (which suggests it might not even be usable).
Regarding the interface, I think a light orange color and a small label reading "beta" directly on the module box would look nice. If the idea 💡 is approved, I volunteer for this task 🤩 |
@pascalgrimaud: I am looking for the next issue to work on. Can I start working on this feature? |
Yes you can, @renanfranca but I have more ideas on that. I don't want to limit to 2 states (availabled / disabled). We should use something like a priority (a value between 1 to 5 for example). Alternatively, we could use a ranking system like in manga, such as Rank S, Rank A, Rank B, etc., to determine which modules to display. So we can choose to display what we want:
As you can see in this screen: There are too many modules now, and some are not used frequently. We should have the ability to decide whether to display them or not. Maven or Spring Boot must be Rank S |
cc @murdos : what do you think? |
Let's start by small steps:
|
Thanks 👍. Should I implement the beta state, too? |
No need, beta state will be RANK_D |
Honestly I'm not convinced, this seems to be a bit complicated and arbitrary. I liked the original idea of highlighting modules in an experimental/beta state. But If the landscape is overloaded, I think we should explore other solutions with https://github.com/ascelineboullen. |
Maybe let's wait a little bit, after your meet with @ascelineboullen And I agree it's totally arbitrage |
@pascalgrimaud : I've got two ideas that might help: 1 - Add an extended JHipster Lite module that lets you hide modules based on tags. 2 - Or maybe we could add an option on the landscape screen to hide or highlight modules by tags. That setting could be saved in local storage so you don't have to reselect it every time you hit F5. WDYT? |
No need to use an extended JHipster Lite module, I use a specific profile with a lot of modules in the
It could be a search input, like in patches page |
In the picture above, the search bar automatically positions the module in the right place. So, should the input text center the module on the screen? Or should it have a different behavior? My concern is placing the search input text in a way that doesn't interfere with the module's visualization. |
@pascalgrimaud : I created a quick demo of how I think the search bar should work (ignore the search button, which wasn't needed). WDYT? edit: the video is lagging, but it's smoothly search_bar_demo.mp4 |
Great @renanfranca ! |
Thanks! I will create a new issue for that ;) |
For me, the idea of module rank is good. I think ranks should go on landscape screen and we can remove replay. An idea to rank modules:
Here production product is a product that's not a toy project and actually meant for end users. I also agree that we should rework the landscape. I build that thing on a 34" monitor with way less module than we have now. The whole thing became totally irrelevant. I think this should be done in a mob session and this is probably a prerequisite for that ranking feature. |
Today, we have 2 states for modules:
I want to add a new state:
These modules will be displayed with a warning or another color (any suggestions are welcomed)
So we can start new modules and get some feedbacks, and totally rework if needed, without worrying about the use
What do you think?
Good or bad idea?
The text was updated successfully, but these errors were encountered: