pop up windows should try to have no scrollbars

Bug #892248 reported by Ferdinand
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Odoo Web (MOVED TO GITHUB)
Fix Released
Wishlist
OpenERP R&D Web Team

Bug Description

IMHO pop up windows should not have scrollbars if enough space is available
does it work with min-height / min-width ?

Related branches

Revision history for this message
Ferdinand (office-chricar) wrote :
Revision history for this message
Vishal Parmar(Open ERP) (vpa-openerp) wrote :

Hello Dr.Ferdinand,

Yes, It should be but it's depend on height and width of the field. In your given example state field is last in pupup so scrollbar is show. I have also see in invoice and PO popup as expected. would you please see attached video.

Thanks and waiting for your reply.

Revision history for this message
Vishal Parmar(Open ERP) (vpa-openerp) wrote :
Changed in openerp-web:
status: New → Incomplete
Revision history for this message
Ferdinand (office-chricar) wrote :

hello
yes it's exactly the problem,
my point is, can the size of the pupup window be dynamic ?

on bigger screens there is no reason to make the pupup to small to be able to view all elements hence I suggest a minimum width instead of absolute width
see
http://www.w3schools.com/cssref/pr_dim_min-height.asp
http://www.w3schools.com/cssref/pr_dim_min-width.asp

Changed in openerp-web:
status: Incomplete → New
Revision history for this message
Vishal Parmar(Open ERP) (vpa-openerp) wrote :

Hello Dr.Ferdinand,

In current code of web(6.1) Size of pupup window is dynamic but It's depends on the screen resolution.

As per my opinion pupup is correctly open and it's not issue.

Thanks and waiting for your reply

Revision history for this message
Ferdinand (office-chricar) wrote :

web revno 1518 of today
please see attachment

I do not see why the popup needs a scrollbar, there is plenty of vertical space.

may be the dynamic calculation must be revised.

Revision history for this message
Vishal Parmar(Open ERP) (vpa-openerp) wrote :

Hello,

Yes I am agree with you regarding thepopup needs a scroll-bar plenty of vertical space.
currently I can consider this issue as wishlist because this needs more clarification.

Thanks and waiting for your reply.

Changed in openerp-web:
assignee: nobody → OpenERP R&D Web Team (openerp-dev-web)
importance: Undecided → Wishlist
status: New → Confirmed
Revision history for this message
Fabien Meghazi (OpenERP) (fme) wrote :

Fixed in Revision:
1806 revid:<email address hidden>

Changed in openerp-web:
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.