Skip to content
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

[18.0] web_responsive and main page apps #2984

Open
bealdav opened this issue Nov 8, 2024 · 12 comments
Open

[18.0] web_responsive and main page apps #2984

bealdav opened this issue Nov 8, 2024 · 12 comments

Comments

@bealdav
Copy link
Member

bealdav commented Nov 8, 2024

Thanks for your fantastic work on responsive module

Just a question.

main page app

image

community drop down app list

image

Describe the solution you'd like
Related to the migration of web_responsive in 18.0.
This module is complex to migrate because of many features.

When a new version arrives it could be good to have a main app page earlier in the migration process to demonstrate to new customers that community version is nice.

Then is it possible to extract the main page app with nice icons from web_responsive module, because you may want to have only this main page apps. Months later you might want responsive features without this main page app with only community drop down app according to your choice.

The drop down is fine to directly select your menu in a productive way, while the main page let your eyes spread anywhere on the page to find the right app.
Having the choices is good.

I probably forgot contributors here, please alert them

cc @yajo @legalsylvain @etobella @Tardo @angelmoya @chienandalu @SplashS

@legalsylvain
Copy link
Contributor

legalsylvain commented Nov 8, 2024

Hi. context : I never worked on web_responsive and I have limited skills in Javascript / CSS.

In fact, I don't understand in which case we want the community drop down list.

  • I see it very poor
  • there is no possibility to search, so it requires more click
  • I don't see it very beautiful. (matter of taste on that topic).
  • maintain 2 (or more) modules is more work in each migration.

So I'm in favor to keep this module as it.

Basically this module takes part of the "must have" modules and tools that should be migrated each year by the OCA. I prefer to work on the topic of how the OCA sustains financialy that work (cf talk in the OCA days).

My two cents.

@PaulGoubert
Copy link

I agree with David’s vision and understand Sylvain’s point: web_responsive is a must-have.

From my perspective, the web_responsive module includes the main menu with icons, which is nice and sexy for new users. It mirrors the way they use apps on their phones.

When a new version is released, it makes sense to base new projects on that version and enhance the UI with this more visual menu, thanks to the icons.

The web_responsive module is comprehensive and includes all the responsiveness features needed for use on tablets, for instance. However, this isn’t necessarily critical in the early months of a new project.

On the other hand, the web interface for the homepage menu allows us to present a more user-friendly side of the ERP during new user training sessions. It feels less intimidating than the dropdown menu of the CE version.

Having a module like web_app_menu or something similar that focuses solely on this feature would be a great addition. It might also be easier to migrate between versions while waiting for the full web_responsive module, with all its features, to become available in the months following a new release.

I have a client who might be interested and potentially willing to partially fund this development.

Again, this is purely cosmetic, but it makes sense from a marketing perspective for our CE version.

@Tardo
Copy link
Member

Tardo commented Nov 26, 2024

It's been a few years since I've been involved in this module.
I don't know in which version the side menu behavior was separated. I think the app menu feature is a good candidate to be separated as well. It goes beyond “responsive” design... It offers a functionality
I think that a module with concrete functions helps to get other developers involved. Working with a “big” module always gives more 'laziness' to get involved. At least that's how I see it.

@chienandalu
Copy link
Member

I might agree with the splitting idea 😄

@pedrobaeza
Copy link
Member

pedrobaeza commented Nov 27, 2024

I prefer to have it together. At the end, we will always install both. Which case do you need one and not the other?

@houssine78
Copy link

I also agree having it spitted. But mostly to have the icons menu available quickly.

My two cents on this topic.

@yajo
Copy link
Member

yajo commented Nov 29, 2024

FWIW, if you hit Ctrl+K and type / + the menu you want, you can search for it without this module in Odoo 16 CE. Not tested in 18.

@bealdav
Copy link
Member Author

bealdav commented Nov 29, 2024

Thanks @yajo for the clarification.

@bealdav
Copy link
Member Author

bealdav commented Nov 29, 2024

@pedrobaeza The topic @PaulGoubert raises is not his/our preferences, it only concerns prospects' preferences.
When a new version of Odoo arrives, @PaulGoubert tries to convince new potential customers to bet on Odoo's Community OCA ecosystem.
He is frequently in front of other partners who sell and demonstrate the Odoo enterprise with a nice landing page. This may be a detail for you, but for these people (prospects), such a landing page can be decisive for their choice.

We are committed to offering the latest version of Odoo when it is released. Then, @PaulGoubert proposes a solution to make the OCA offer attractive a few weeks after the release of a new version. And he offers to contribute financially to an OCA expert to make such a module easier and faster to migrate.

I see that people's reactions tend to be positive to his proposal.

And yet, reducing the scope of a module is good for quality and makes migration easier.

[EDIT] ... and make subset features migration earlier .

@pedrobaeza
Copy link
Member

As said by @yajo: you don't need menu apps for searching menus and having a graphical interface. Just use command pallette with the / character. The menu app is part of the responsive experience for having a touching type display, so it shouldn't be dettached IMO.

@pedrobaeza
Copy link
Member

And yet, reducing the scope of a module is good for quality and makes migration easier.

No, this is not true a lot of times. Decoupling things require to create interfaces, hooks, etc for extensions to attach there, and the maintenance burden for keeping everything compatible is sometimes even more (and it always requires more lines of code).

@legalsylvain
Copy link
Contributor

@SplashS : could you provide your point of view on this topic ? you are the second developper marked as "maintainers" in the latest released version. https://github.com/OCA/web/tree/17.0/web_responsive#maintainers
(@Tardo already gave an opinion here : #2984 (comment))
Thanks !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

8 participants