changeset 7246:b7698ff4b6e3

[Accessibility] Pagination is not annoucing selected page changed (using keyboard). This happens because the notification was sent before the selected page changed. Thus, at the time the notification windows 8 see nothing really changed and annouces nothing, Windows 7 annouces the old (and wrong) page. The fix is to send the notification after the selected page changes.
author Felipe Heidrich <felipe.heidrich@oracle.com>
date Wed, 11 Jun 2014 12:09:58 -0700
parents 9229486ecaea
children 6eaeae29f418
files modules/controls/src/main/java/com/sun/javafx/scene/control/skin/PaginationSkin.java modules/controls/src/main/java/javafx/scene/control/Pagination.java
diffstat 2 files changed, 1 insertions(+), 2 deletions(-) [+]
line wrap: on
line diff
--- a/modules/controls/src/main/java/com/sun/javafx/scene/control/skin/PaginationSkin.java	Wed Jun 11 12:06:59 2014 -0700
+++ b/modules/controls/src/main/java/com/sun/javafx/scene/control/skin/PaginationSkin.java	Wed Jun 11 12:09:58 2014 -0700
@@ -854,6 +854,7 @@
                     break;
                 }
             }
+            getSkinnable().accSendNotification(Attribute.SELECTED_PAGE);
         }
 
         // Update the page index using the currentIndex and updates the page set
--- a/modules/controls/src/main/java/javafx/scene/control/Pagination.java	Wed Jun 11 12:06:59 2014 -0700
+++ b/modules/controls/src/main/java/javafx/scene/control/Pagination.java	Wed Jun 11 12:09:58 2014 -0700
@@ -254,8 +254,6 @@
                     setCurrentPageIndex(getPageCount() - 1);
                 }
             }
-
-            accSendNotification(Attribute.SELECTED_PAGE);
         }
 
         @Override