josephsl/goldencursor

Golden Cursor NvDA add-on

goldencursor's Issues

Positions database: migrate to a proper dictionary format via ConfigObj

Hi,

Currently, .gc (position listings) files are stored as plain text files. at first glance, this is a good approach as the add-on comes with positions parser. However, in the long run, this format isn't viable.

Thus it is proposed that the positions database be moved to a more dictionary-like interface, preferably via ConfigObj. This means:

• Familiar key/value pairs structure for each file.
• If possible, no more string conversions.
• Familiar ini syntax.
• No more cursor parsers required, as the dictionary will take care of this.

Costs:

• Backward compatibility: due to change from text files to a dictionary, backward compatibility is not guaranteed. One way to fix this is to provide a new folder just for 2.0 config and a migration routine. Note that this is still a one-way migration.

Thanks.

Saving positions: file not found error when files folder does not exist

Hi,

When trying to save positions, NVDA says, "file not found".

STR:

1. Try saving a position (NvDA+Shift=L) for an app.
2. Type position coordinates and press Enter.

Expected: NVDA saves positions.
Actual:

ERROR - unhandled exception (08:31:09.701):
Traceback (most recent call last):
File "wx_misc.pyc", line 1367, in Notify
File "wx_core.pyc", line 16869, in Notify
File "C:\Users\User\AppData\Roaming\nvda\addons\goldenCursor\globalPlugins\goldenCursor.py", line 201, in saving
File "codecs.pyc", line 896, in open
IOError: [Errno 2] No such file or directory: path

Cause: the add-on assumes a folder named "files" already exists.
Solution: catch this case and create a new directory.

Thanks.

There is a string that cannot be translated in GC5.0

On line 348, it can be translated if it is changed to the following:
gui.mainFrame, _("Enter the name for the current mouse position (x: {x}, Y: {y})").format(x=x, y=y),

Remove win32api dependency

Hi,

As NVDA Core does not use win32api, it is fitting that this add-on remove this dependency. With this gone, NVDA Core can officially remove its own dependence on win32api.

Thanks.