Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
0010566Openbravo ERP09. Financial managementpublic2009-09-11 15:312009-10-22 00:00
networkb 
dbaz 
highminoralways
closedfixed 
5
2.40MP8 
2.40MP112.50MP7 
Core
No
0010566: GeneralLedgerReport changes its focus when clicking in any part of the window.
When you run GeneralLedgerReport in html format, if you scrolls down or scrolls right to check data obtained and then clicks in any part of the window, the window scrolls up automatically to focus's place
1.Financial Management || Accounting || Analysis Tools || General Ledger Report:
    1.1. Choose any valid date
    1.2. Run report
2. Scrolls down to check any data and click in any part of the window
3. Check that you loose your position and the window goes up to focus place
No tags attached.
depends on backport 00105692.40MP11 closed dbaz GeneralLedgerReport changes its focus when clicking in any part of the window. 
Issue History
2009-09-11 15:31networkbNew Issue
2009-09-11 15:31networkbAssigned To => rafaroda
2009-09-11 15:39rafarodaAssigned Torafaroda => dalsasua
2009-09-11 15:39rafarodaPriorityimmediate => high
2009-09-11 15:39rafarodaStatusnew => scheduled
2009-09-11 15:39rafarodafix_in_branch => pi
2009-09-11 15:49rafarodaNote Added: 0019808
2009-09-11 15:49rafarodaAssigned Todalsasua => dbaz
2009-09-11 15:49rafarodafix_in_branchpi =>
2009-09-11 22:47dbazNote Added: 0019835
2009-09-11 22:47dbazStatusscheduled => feedback
2009-09-21 18:57networkbNote Added: 0020247
2009-09-22 16:29networkbStatusfeedback => new
2009-09-25 07:25rafarodaStatusnew => acknowledged
2009-09-28 14:43hgbotCheckin
2009-09-28 14:43hgbotNote Added: 0020405
2009-09-28 14:43hgbotStatusacknowledged => resolved
2009-09-28 14:43hgbotResolutionopen => fixed
2009-09-28 14:43hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/6cca33b96d30effc65317278101d1a6443f0c340 [^]
2009-09-29 14:19hgbotCheckin
2009-09-29 14:19hgbotNote Added: 0020521
2009-09-29 14:19hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/devel/pi/rev/6cca33b96d30effc65317278101d1a6443f0c340 [^] => http://code.openbravo.com/erp/devel/pi-pageddatagrid/rev/6cca33b96d30effc65317278101d1a6443f0c340 [^]
2009-10-21 11:17arunkumarStatusresolved => closed
2009-10-21 11:17arunkumarNote Added: 0021254
2009-10-21 11:17arunkumarFixed in Version => 2.50MP7
2009-10-22 00:00anonymoussf_bug_id0 => 2883558

Notes
(0019808)
rafaroda   
2009-09-11 15:49   
David,

Is it a problem with focus function or something wrongly coded in this HTML?

Thanks.
(0019835)
dbaz   
2009-09-11 22:47   
I don't see where the problem is...

I mean, due to OB focus logic, just some items can have focus, and this is a constrain that fight agains browser default focus behavior.

To success in that behaviour, the JS checks where a click is done to see if it can have focus or not. If not, the focus return to the last permitted focused item.

Desglosing the behavior:
1. Item A has focus
2. You click on a item B which can not have focus following OB rules
3. Browser default focus behavior sets the focus to this element B (this can not be jumped nor faked)
4. OB javascript check that B could not have focus
5. OB javascript put focus back on element A

Now what is happening is that you click in a part of the form where there is not element that can have focus (following OB focus logic), so the focus return to the last focused item. If this last focused item is no visible, the scroll moves to a position where it is visible (it is a standard browser behavior).

Once more time, I don't know what is your problem or your client's problem with this behavior.
(0020247)
networkb   
2009-09-21 18:57   
Ok, so in order to justify this behavior to our partner, could you please provide the link or the documentation where OB focus logic is explained to share it with him.

Regards
(0020405)
hgbot   
2009-09-28 14:43   
Repository: erp/devel/pi
Changeset: 6cca33b96d30effc65317278101d1a6443f0c340
Author: David Baz Fayos <david.baz <at> openbravo.com>
Date: Mon Sep 28 13:29:50 2009 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/6cca33b96d30effc65317278101d1a6443f0c340 [^]

Fixed issue 10566: Changed focus behavior when clicking on empty spaces

---
M src/org/openbravo/erpCommon/security/Login_F1.html
M web/js/utils.js
M web/js/windowKeyboard.js
---
(0020521)
hgbot   
2009-09-29 14:19   
Repository: erp/devel/pi-pageddatagrid
Changeset: 6cca33b96d30effc65317278101d1a6443f0c340
Author: David Baz Fayos <david.baz <at> openbravo.com>
Date: Mon Sep 28 13:29:50 2009 +0200
URL: http://code.openbravo.com/erp/devel/pi-pageddatagrid/rev/6cca33b96d30effc65317278101d1a6443f0c340 [^]

Fixed issue 10566: Changed focus behavior when clicking on empty spaces

---
M src/org/openbravo/erpCommon/security/Login_F1.html
M web/js/utils.js
M web/js/windowKeyboard.js
---
(0021254)
arunkumar   
2009-10-21 11:17   
Tested & Working fine.