Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0016182Openbravo ERPB. User interfacepublic2011-03-09 12:172011-11-22 03:50
Reporterplujan 
Assigned Todbaz 
PrioritynormalSeveritymajorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Versionpi 
Target VersionFixed in Version3.0MP6 
Merge Request Status
Review Assigned To
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0016182: Heartbeat Log tab does not scroll the "installed modules" field
DescriptionThe Heartbeat Log tab is by default disabled, meaning that all its fields are read only. However, the "installed modules" field's content required scrolling, what is not possible in a disabled field. See attach.
Steps To ReproduceUsing Firefox:
* Go to "Heartbeat Configuration"
* Go to its child tab "Heartbeat Log"
* Realize that "Installed Modules" field can not be scrolled
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
related to feature request 0018411 new dbaz Read only form fields can not be selected/highlighted using Firefox 
Attached Filespng DisableFieldDoesNotScroll.PNG (22,993) 2011-03-09 12:17
https://issues.openbravo.com/file_download.php?file_id=3639&type=bug
png
Issue History
Date ModifiedUsernameFieldChange
2011-03-09 12:17plujanNew Issue
2011-03-09 12:17plujanAssigned To => dbaz
2011-03-09 12:17plujanFile Added: DisableFieldDoesNotScroll.PNG
2011-03-09 12:17plujanModules => Core
2011-03-10 18:31dbazAssigned Todbaz => mtaal
2011-03-14 10:56alostaleStatusnew => scheduled
2011-03-15 09:53hgbotCheckin
2011-03-15 09:53hgbotNote Added: 0034856
2011-03-16 11:23mtaalNote Added: 0034926
2011-09-07 13:34mtaalAssigned Tomtaal => dbaz
2011-10-18 01:20dbazRelationship addedrelated to 0018411
2011-10-18 01:32hgbotCheckin
2011-10-18 01:32hgbotNote Added: 0041868
2011-10-18 01:40dbazNote Added: 0041869
2011-10-18 08:07mtaalNote Added: 0041871
2011-10-23 13:41hudsonbotCheckin
2011-10-23 13:41hudsonbotNote Added: 0042011
2011-11-17 16:57jmonIssue Monitored: jmon
2011-11-17 17:03jmonNote Added: 0042947
2011-11-17 21:55dbazSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=2914#r2914
2011-11-17 21:56dbazNote Added: 0042961
2011-11-17 21:59hgbotCheckin
2011-11-17 21:59hgbotNote Added: 0042962
2011-11-17 21:59hgbotStatusscheduled => resolved
2011-11-17 21:59hgbotResolutionopen => fixed
2011-11-17 21:59hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/145e35bc7e5b86aa52163c21117790601a485b37 [^]
2011-11-20 23:38mtaalNote Added: 0043002
2011-11-20 23:38mtaalStatusresolved => closed
2011-11-20 23:38mtaalFixed in Version => 3.0MP6
2011-11-22 03:50hudsonbotCheckin
2011-11-22 03:50hudsonbotNote Added: 0043051

Notes
(0034856)
hgbot   
2011-03-15 09:53   
Repository: erp/devel/pi
Changeset: d725db400e1fcfea6d6eb8bbbecb068d3652e6c2
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Tue Mar 15 09:51:07 2011 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/d725db400e1fcfea6d6eb8bbbecb068d3652e6c2 [^]

Related to issue 16182: Heartbeat Log tab does not scroll disabled textarea field, add hover for disabled text fields

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/ob-formitem-widgets.js
---
(0034926)
mtaal   
2011-03-16 11:23   
Asked on the smartclient forum, got a reply saying that we should try the readonly property:
http://forums.smartclient.com/showthread.php?t=15941 [^]

Will do this after unfreeze of pi
(0041868)
hgbot   
2011-10-18 01:32   
Repository: erp/devel/pi
Changeset: a6c57b71ef31601c4479539f04f2069b901e100b
Author: David Baz Fayos <david.baz <at> openbravo.com>
Date: Tue Oct 18 01:32:11 2011 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/a6c57b71ef31601c4479539f04f2069b901e100b [^]

Related to issue 16182: disabled textarea set again

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/formitem/ob-formitem-textarea.js
---
(0041869)
dbaz   
2011-10-18 01:40   
The problem is that textarea is set as disabled, and today still is not available/supported a "readonly" form item in Smartclient.

So by the moment we can only have the html standard "disable" feature, where:

In IE you can select the text and go to the end.
In Chrome you can select the text and also operate the scrolls.
In Firefox you can not select nor operate the scrolls.

Thanks to mtaal's push, you can hover a disabled text area to show a promt with its content.
(0041871)
mtaal   
2011-10-18 08:07   
Smartclient has a readOnly property on the TextAreaItem. In addition there are several posts on the smartclient forum discussing how to solve this for other form items without this property.

The main issue is that for a form the disabled property is used to signal that a textitem can't be changed. So in this case we need an additional way of keeping track that the textareaitem is actually disabled (i.e. that it is readonly).

Can you test with the readOnly on text area items? Also can you check the forum posts on this topic and the smartclient source code?

gr. Martin
(0042011)
hudsonbot   
2011-10-23 13:41   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/35bf0a7aa166 [^]

Maturity status: Test
(0042947)
jmon   
2011-11-17 17:03   
Hi, I need to use a text box in a read-only tab but I have this problem of not being able to scroll to see the content as you describe in this issue.

I see the changeset has been in Test status and not included in mp5.
Do you have any plans on this?

I should decide wether to wait for the fix or use as work around such as setting the tab as editable and preventing changes with a trigger.
(0042961)
dbaz   
2011-11-17 21:56   
To perform a test, follow the "Steps To Reproduce"
(0042962)
hgbot   
2011-11-17 21:59   
Repository: erp/devel/pi
Changeset: 145e35bc7e5b86aa52163c21117790601a485b37
Author: David Baz Fayos <david.baz <at> openbravo.com>
Date: Thu Nov 17 21:59:00 2011 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/145e35bc7e5b86aa52163c21117790601a485b37 [^]

Fixed issue 16182: Now disabled textareas can be scrolled in Firefox

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/formitem/ob-formitem-textarea.js
---
(0043002)
mtaal   
2011-11-20 23:38   
Reviewed
(0043051)
hudsonbot   
2011-11-22 03:50   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/ebd713dfe507 [^]

Maturity status: Test