Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

an irrelevant msg is given on opening manage rules and alerts dialog in Outlook 2013. #3407

Closed
nvaccessAuto opened this issue Aug 5, 2013 · 8 comments
Labels
Milestone

Comments

@nvaccessAuto
Copy link

@nvaccessAuto nvaccessAuto commented Aug 5, 2013

Reported by sumandogra on 2013-08-05 09:41
Outlook 2013

MSO generic controlcontainer… an irrelevant msg is given onopening the manage alerts and rules dialog.
Steps to reproduce:

  1. Go to the Home tab and ARROW to rules.
  2. Select manage alerts and rules from the rules grouping.
  3. The dialog opens.
  4. NVDA gives an irrelevant “mso generic control….”before announcing the focused controlin the dialog.

Blocked by #4096

@nvaccessAuto
Copy link
Author

@nvaccessAuto nvaccessAuto commented Apr 21, 2014

Comment 2 by Joseph Lee <joseph.lee22590@... on 2014-04-21 03:45
In [c984b60129f6e9ca82956bcd79f54eb383761e4f]:

Unnecessary toolbar names are ignored in Outlook message rules dialog. Fixes #3407.
In message rules dialog, MsoCommandBar toolbar was kept announced since it was the parent of controls such as new and copy buttons. Worked out a hack to assign no name and description to this toolbar (one can still obtain the container name via window text). To further prevent toolbar announcement, this object has been reassigned to the role of window.
Fix for #3407 (hope there is an even better solution than this hack).

Changes:
State: closed

@nvaccessAuto
Copy link
Author

@nvaccessAuto nvaccessAuto commented Apr 21, 2014

Comment 3 by nvdakor on 2014-04-21 03:50
Hi jteh and mdcurran,
Sorry about the ticket changes - it was done by accident. It should not be closed, owner should not be set to me and what not (again sorry about that).

@nvaccessAuto
Copy link
Author

@nvaccessAuto nvaccessAuto commented Apr 21, 2014

Comment 4 by jteh on 2014-04-21 07:52
Changes:
State: reopened

@nvaccessAuto
Copy link
Author

@nvaccessAuto nvaccessAuto commented Apr 28, 2014

Comment 5 by mdcurran on 2014-04-28 05:36
Technically, most of the controls in that dialog are within that toolbar. I don't think we should remove it completely from the focus ancestory as it is a relevant grouping, but, assuming that its named this way no mater the localization of Outlook (which seems possible due to the strange naming) then we can just ditch the name, and desciption if theya re named like this. I'll do it on t4096 branch as it requires that class anyway.

@nvaccessAuto
Copy link
Author

@nvaccessAuto nvaccessAuto commented Apr 28, 2014

Comment 7 by Michael Curran <mick@... on 2014-04-28 05:46
In [4c3beed]:

Merge branch 't3407' into next. Incubates #3407

Changes:
Added labels: incubating

@nvaccessAuto
Copy link
Author

@nvaccessAuto nvaccessAuto commented May 1, 2014

Comment 8 by jteh on 2014-05-01 00:23
Changes:
Milestone changed from None to next

@nvaccessAuto
Copy link
Author

@nvaccessAuto nvaccessAuto commented Jun 16, 2014

Comment 10 by Michael Curran <mick@... on 2014-06-16 01:22
In [3da031c]:

Merge branch 't3407'. Fixes #3407

Changes:
Removed labels: incubating
State: closed

@nvaccessAuto
Copy link
Author

@nvaccessAuto nvaccessAuto commented Jun 16, 2014

Comment 11 by mdcurran on 2014-06-16 01:23
Changes:
Milestone changed from next to 2014.3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant