[MGNLUI-1011] Form Validation: 'Jump to next error' does not work Created: 28/Mar/13 Updated: 25/Jun/13 Resolved: 17/Apr/13 |
|
| Status: | Closed |
| Project: | Magnolia UI |
| Component/s: | forms |
| Affects Version/s: | 5.0 |
| Fix Version/s: | 5.0 |
| Type: | Bug | Priority: | Neutral |
| Reporter: | Christopher Zimmermann | Assignee: | Aleksandr Pchelintcev |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Template: |
|
||||||||
| Acceptance criteria: |
Empty
|
||||||||
| Task DoD: |
[ ]*
Doc/release notes changes? Comment present?
[ ]*
Downstream builds green?
[ ]*
Solution information and context easily available?
[ ]*
Tests
[ ]*
FixVersion filled and not yet released
[ ] 
Architecture Decision Record (ADR)
|
||||||||
| Bug DoR: |
[ ]*
Steps to reproduce, expected, and actual results filled
[ ]*
Affected version filled
|
||||||||
| Date of First Response: | |||||||||
| Description |
|
Clicking on Jump to next error at the top of a form that has some validation errors does not cause anything to happen. |
| Comments |
| Comment by Christopher Zimmermann [ 28/Mar/13 ] |
|
Feature was completely broken because the view no longer had a reference to the presenter. Now the presenter adds itself to the view in FormConnector.init(). Plus 2 logic errors in functionality. 2. Previously it only searched the number of tabs in the form - must search another tab - the first one AGAIN, but this time from the top. This is necessary for the search to wrap to the current tab when errors are only on one tab. |
| Comment by Aleksandr Pchelintcev [ 05/Apr/13 ] |
|
The feature still has several glitches, e.g. the red box does not disappear and count of errors isn't updated properly sometimes. |
| Comment by Christopher Zimmermann [ 17/Apr/13 ] |
|
Sasha - that is a different error: http://jira.magnolia-cms.com/browse/MGNLUI-1044 (Form Validation: Error Header does not stay in sync with state of form) This ticket is just about the "jump to next error" link working. |