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

MS Word table navigation scripts rather slow #3326

Closed
nvaccessAuto opened this issue Jul 3, 2013 · 4 comments
Closed

MS Word table navigation scripts rather slow #3326

nvaccessAuto opened this issue Jul 3, 2013 · 4 comments
Assignees
Labels
Milestone

Comments

@nvaccessAuto
Copy link

Reported by mdcurran on 2013-07-03 05:01
Table navigation scripts in MS Word (control+alt+arrows) are rather slow, especially in large tables.
It looks like the calls to range.information to get row and column numbers is what takes the most time. In deed any call to range.information has always been bad, so we should strive not to use it where ever possible.
MS Word cell objects have rowIndex and columnIndex properties (2003 and up) which seem to be much faster, so these should be used instead.

@nvaccessAuto
Copy link
Author

Comment 1 by mdcurran on 2013-07-03 05:20
Try build here:
http://community.nvda-project.org/try/t3326/nvda_snapshot_try-t3326-9283,829adba.exe
I tested with Word 2010 and can test 2007, but need others to test on 2003 if possible.
The t3326 branch appart from not using range.information also improves the table movement algorithm some what to make less calls to Word and also fixes an issue when trying to move down a column with cells that span multiple rows.

@nvaccessAuto
Copy link
Author

Comment 2 by mdcurran on 2013-07-08 06:22
Incubating in 2def69a
Changes:
Added labels: incubating

@nvaccessAuto
Copy link
Author

Comment 3 by Michael Curran <mick@... on 2013-07-22 00:32
In [3b61417]:

Merge branch 't3326' - Performance improvements when using table navigation commands in MS Word

Fixes #3326

Conflicts:
    user_docs/en/changes.t2t

Changes:
Removed labels: incubating
State: closed

@nvaccessAuto
Copy link
Author

Comment 4 by mdcurran on 2013-07-22 03:41
Changes:
Milestone changed from next to 2013.2

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

No branches or pull requests

2 participants