+
Skip to content

Conversation

parvezqureshi
Copy link

@parvezqureshi parvezqureshi commented Mar 12, 2025

The attachments of customer invoices appear automatically (see screenshot). When we install oca/web/web_responsive, the attachments do not appear automatically (see screenshot). This is on a computer screen of course.

@OCA-git-bot
Copy link
Contributor

Hi @Tardo, @SplashS,
some modules you are maintaining are being modified, check this out!

@parvezqureshi parvezqureshi force-pushed the 17.0-fix-web_responsive branch from fd7d4bc to d4a58bf Compare March 12, 2025 07:13
Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is done on purpose. We don't want the preview to appear automatically. See #2745 #2795.

If you want to enable it, please include a system or user option.

@pedrobaeza pedrobaeza added this to the 17.0 milestone Mar 12, 2025
@parvezqureshi
Copy link
Author

@cvinh please review.

@cvinh
Copy link
Contributor

cvinh commented Mar 12, 2025

@cvinh please review.

Thanks @parvezqureshi please include a setup by user if someone wants attachment to open automatically or not... default =False (thanks to @pedrobaeza )

Please remove the screenshots, there are personal data in them... I think the reviewers understood the purpose

@parvezqureshi parvezqureshi force-pushed the 17.0-fix-web_responsive branch 6 times, most recently from 7fa550c to 19d2832 Compare March 12, 2025 09:18
<data>
<field name="action_id" position="after">
<field name="is_redirect_home" invisible="action_id" />
<field name="display_file_viewer" />
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems it has an incorrect indentation

Copy link
Contributor

@cvinh cvinh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please can you :

  • improve the README explaining this new functionality
  • change your commit comment with [IMP] instead of [FIX]

store=True,
readonly=False,
)
display_file_viewer = fields.Boolean()
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add a help to explain what this field does

@cvinh
Copy link
Contributor

cvinh commented Mar 12, 2025

cc @vehi-invitu

Copy link

@vehi-invitu vehi-invitu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The setup is missing in base.view_users_form_simple_modif view

@hbrunn, what do you about this setup user ? It should be good solution for your PR
#3058

@parvezqureshi parvezqureshi force-pushed the 17.0-fix-web_responsive branch from 19d2832 to cabeea5 Compare March 13, 2025 05:17
@parvezqureshi parvezqureshi force-pushed the 17.0-fix-web_responsive branch from cabeea5 to 725185e Compare March 13, 2025 05:28
Copy link
Contributor

@cvinh cvinh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@cvinh cvinh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok for me

Copy link

@vehi-invitu vehi-invitu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the changes. It's good for me !

@jcdrubay
Copy link

jcdrubay commented Apr 22, 2025

In the search view, from google chrome on desktop, when rendering the page, the selection field for Custom Group By as rendering with very small font-size (even after refresh)

http://oca-web-17-0-pr3122-725185e11078.runboat.odoo-community.org/web?reload=true#action=69&model=res.users&view_type=kanban&menu_id=60&cids=1

image

@cvinh
Copy link
Contributor

cvinh commented May 14, 2025

In the search view, from google chrome on desktop, when rendering the page, the selection field for Custom Group By as rendering with very small font-size (even after refresh)

http://oca-web-17-0-pr3122-725185e11078.runboat.odoo-community.org/web?reload=true#action=69&model=res.users&view_type=kanban&menu_id=60&cids=1

image

@parvezqureshi can you check this please ?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载