Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0023853Openbravo ERPA. Platformpublic2013-05-20 11:202014-02-28 19:50
Reportercaristu 
Assigned Todbaz 
PriorityhighSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target Version3.0MP26Fixed in Version3.0MP26 
Merge Request Status
Review Assigned ToAugustoMauch
OBNetwork customerOBPS
Web browser
ModulesCore
Support ticket21232
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0023853: Rich Text fields are not working properly in some browsers
DescriptionRich Text Fields are not working properly in Chrome and Firefox. Make the font bold is not working fine (see attachment) and window doesn't detect any change when editing the line, is necessary to click on a button of the rich text field in case you want to save the changes.
Steps To ReproduceAs system administrator
1) Create a new column using rich text field reference.
2) Create a field for the column created in step 1
As group admin role
3) Use the rich text field for making a text bold
4) Edit only the text in the record and notice that the save button remains as not clickable.
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
related to defect 0022933 closed dbaz Keypress and tab issue in rich text field 
Attached Filespng RichTextField.png (7,176) 2013-05-20 11:21
https://issues.openbravo.com/file_download.php?file_id=6147&type=bug
png
Issue History
Date ModifiedUsernameFieldChange
2013-05-20 11:20caristuNew Issue
2013-05-20 11:20caristuAssigned To => AugustoMauch
2013-05-20 11:20caristuModules => Core
2013-05-20 11:20caristuOBNetwork customer => Yes
2013-05-20 11:20caristuSupport ticket => 21232
2013-05-20 11:20caristuResolution time => 1375394400
2013-05-20 11:20caristuTriggers an Emergency Pack => No
2013-05-20 11:20caristuRelationship addedrelated to 0022933
2013-05-20 11:20caristuIssue Monitored: networkb
2013-05-20 11:21caristuFile Added: RichTextField.png
2013-05-20 18:55roklenardicIssue Monitored: roklenardic
2013-06-10 11:53shankarbNote Added: 0059292
2013-06-10 11:53shankarbTypedefect => design defect
2013-06-10 11:55AugustoMauchTarget Version3.0MP25 => 3.0MP26
2013-06-28 15:17caristuNote Added: 0059778
2013-07-02 17:53AugustoMauchAssigned ToAugustoMauch => dbaz
2013-07-05 00:07dbazIssue Monitored: AugustoMauch
2013-07-05 00:07dbazReview Assigned To => AugustoMauch
2013-07-05 00:21dbazNote Added: 0059877
2013-07-05 00:22hgbotCheckin
2013-07-05 00:22hgbotNote Added: 0059878
2013-07-05 00:22hgbotStatusnew => resolved
2013-07-05 00:22hgbotResolutionopen => fixed
2013-07-05 00:22hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/f61d2d0da2fa7144c4bf070874d85d935dd14c96 [^]
2013-07-05 14:15AugustoMauchNote Added: 0059896
2013-07-05 14:15AugustoMauchStatusresolved => new
2013-07-05 14:15AugustoMauchResolutionfixed => open
2013-07-05 18:18dbazNote Added: 0059906
2013-07-05 21:12dbazNote Edited: 0059906bug_revision_view_page.php?bugnote_id=0059906#r4856
2013-07-05 21:13hgbotCheckin
2013-07-05 21:13hgbotNote Added: 0059907
2013-07-05 21:13hgbotStatusnew => resolved
2013-07-05 21:13hgbotResolutionopen => fixed
2013-07-05 21:13hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/devel/pi/rev/f61d2d0da2fa7144c4bf070874d85d935dd14c96 [^] => http://code.openbravo.com/erp/devel/pi/rev/cba285f50e81ecf7026ba4575f85338c6bf4470c [^]
2013-07-15 11:56AugustoMauchNote Added: 0059968
2013-07-15 11:56AugustoMauchStatusresolved => new
2013-07-15 11:56AugustoMauchResolutionfixed => open
2013-07-15 14:12hgbotCheckin
2013-07-15 14:12hgbotNote Added: 0059971
2013-07-15 14:12hgbotStatusnew => resolved
2013-07-15 14:12hgbotResolutionopen => fixed
2013-07-15 14:12hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/devel/pi/rev/cba285f50e81ecf7026ba4575f85338c6bf4470c [^] => http://code.openbravo.com/erp/devel/pi/rev/de65f6acc7f000c0877b0baa0c15218ed577a635 [^]
2013-07-15 14:45dbazNote Added: 0059972
2013-07-15 16:54AugustoMauchNote Added: 0059986
2013-07-15 16:54AugustoMauchStatusresolved => closed
2013-07-15 16:54AugustoMauchFixed in Version => 3.0MP26
2013-07-16 14:26hudsonbotCheckin
2013-07-16 14:26hudsonbotNote Added: 0060027
2013-07-16 14:26hudsonbotCheckin
2013-07-16 14:26hudsonbotNote Added: 0060032
2013-07-16 14:27hudsonbotCheckin
2013-07-16 14:27hudsonbotNote Added: 0060044
2014-02-26 15:32hgbotCheckin
2014-02-26 15:32hgbotNote Added: 0064615
2014-02-28 19:50hudsonbotCheckin
2014-02-28 19:50hudsonbotNote Added: 0064693

Notes
(0059292)
shankarb   
2013-06-10 11:53   
The RichTextEditor's keydown event does not work properly in chrome. I have added a post in smartclient forums for the same [1].
http://forums.smartclient.com/showthread.php?p=105626#post105626 [^]
(0059778)
caristu   
2013-06-28 15:17   
The styling problem (bold, italic, underlined) seems to be not reproducible if we have the org.openbravo.userinterface.smartclient.dev module installed
(0059877)
dbaz   
2013-07-05 00:21   
For the test case use WITH and WITHOUT "org.openbravo.userinterface.smartclient.dev" module in place and in development (remember also to update it)
(0059878)
hgbot   
2013-07-05 00:22   
Repository: erp/devel/pi
Changeset: f61d2d0da2fa7144c4bf070874d85d935dd14c96
Author: David Baz Fayos <david.baz <at> openbravo.com>
Date: Fri Jul 05 00:21:46 2013 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/f61d2d0da2fa7144c4bf070874d85d935dd14c96 [^]

Fixed issue 23853: Updated RichTextCanvas with latest Smartclient changes
to fix 'keyDown' behavior

---
M modules/org.openbravo.userinterface.smartclient/web/org.openbravo.userinterface.smartclient/isomorphic/ISC_Combined.js
---
(0059896)
AugustoMauch   
2013-07-05 14:15   
This issue is still reproducible: if I update the text of a rich text field, the save button is not enabled until I press another button of the rich text field.

Test case:
- Change the reference of the Description field of the Module window to Rich Text Field.
- Compile, restart tomcat.
- Open the Module window
- Open any record in form view
- Update the text of the description. Notice that the save button is not updated in the toolbar.
(0059906)
dbaz   
2013-07-05 18:18   
(edited on: 2013-07-05 21:12)
There is a cross-domain problem in Chrome, that raises the following JS error

Blocked a frame with origin "http://domainIP:8080" [^] from accessing a frame with origin "http://domainIP.homeip.net:8080". [^] The frame being accessed set "document.domain" to "IP", but the frame requesting access did not. Both must set "document.domain" to the same value to allow access.

To retest the issue please clear the cache.
(0059907)
hgbot   
2013-07-05 21:13   
Repository: erp/devel/pi
Changeset: cba285f50e81ecf7026ba4575f85338c6bf4470c
Author: David Baz Fayos <david.baz <at> openbravo.com>
Date: Fri Jul 05 21:13:30 2013 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/cba285f50e81ecf7026ba4575f85338c6bf4470c [^]

Fixed issue 23853: Added missing 'empty.html' file
needed to avoid 'document.domain' problems in RichTextCanvas

---
A modules/org.openbravo.userinterface.smartclient/web/org.openbravo.userinterface.smartclient/isomorphic/system/helpers/empty.html
---
(0059968)
AugustoMauch   
2013-07-15 11:56   
Even though now it detects properly the addition of new characters, if a character is deleted using the Backspace or the Delete keys the change is not detected and the Save button is not enabled.
(0059971)
hgbot   
2013-07-15 14:12   
Repository: erp/devel/pi
Changeset: de65f6acc7f000c0877b0baa0c15218ed577a635
Author: David Baz Fayos <david.baz <at> openbravo.com>
Date: Mon Jul 15 14:12:21 2013 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/de65f6acc7f000c0877b0baa0c15218ed577a635 [^]

Fixed issue 23853: Now 'Backspace' triggers form change in OBRichTextItem

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/formitem/ob-formitem-richtext.js
---
(0059972)
dbaz   
2013-07-15 14:45   
There are small topics remaining:

http://forums.smartclient.com/showthread.php?p=107034#post107034 [^]
(0059986)
AugustoMauch   
2013-07-15 16:54   
Code reviewed and verified in pi@436cd2fa79fc
(0060027)
hudsonbot   
2013-07-16 14:26   
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/9a5d5983399f [^]

Maturity status: Test
(0060032)
hudsonbot   
2013-07-16 14:26   
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/9a5d5983399f [^]

Maturity status: Test
(0060044)
hudsonbot   
2013-07-16 14:27   
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/9a5d5983399f [^]

Maturity status: Test
(0064615)
hgbot   
2014-02-26 15:32   
Repository: erp/devel/pi
Changeset: b083a207379000136005e486227a1bc342809aac
Author: David Baz Fayos <david.baz <at> openbravo.com>
Date: Tue Sep 10 20:33:58 2013 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/b083a207379000136005e486227a1bc342809aac [^]

[scupgrade] Related to issue 23853: Removed unneeded code
due that latest Smartclient 9.1d implements itself a fix for
the 'Delete' and 'Backspace' keyDown handler

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/formitem/ob-formitem-richtext.js
---
(0064693)
hudsonbot   
2014-02-28 19: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/2a3305b1338e [^]
Maturity status: Test