Accessibility issues #64

Closed
gavinandresen opened this Issue Feb 11, 2011 · 4 comments

Comments

Projects
None yet
3 participants
@gavinandresen
Contributor

gavinandresen commented Feb 11, 2011

I'm opening this for a blind bitcoin user:

There are a couple concerns I have with the accessibility of the GUI in Bitcoin. The "Send Coins" and "Addressbook" buttons are not in the tabstop list, so a blind person with a screen reader cannot tab to and activate these buttons. Also, the current address and balance are also not accessible via pressing tab. When arrowing through the list of transactions, the screen reader, rather than starting out with the status (number of confirmations), reads a long number, usually starting out with 00. If you wait long enough, the rest of the table is read, but unfortunately that is a while to wait.

Also, when bringing up transaction details, there is an html window displayed, but absolutely no feedback is given from the screen reader. Last, in the addressbook, tabbing from control to control works fine, but when shift-tabbing, only the list and tab control are tabbed to.

Of course, most of these concerns, excluding the HTML window which isn't read at all, are not a big deal for myself, as I'm a power user and can use my screen reader's review cursor to read most of the controls. However, there are many blind people who find this method of exploring windows confusing (many will ask "how did you know that was there?"), and these fixes will make it easier for them to explore the Bitcoin interface on their own and use it successfully.

The screen reader I'm using is NVDA, a free and open source screen reader, which can be found at http://www.nvda-project.org.

@ghost

This comment has been minimized.

Show comment Hide comment
@ghost

ghost Feb 12, 2011

May I suggest operating Bitcoin from the command line? The documentation on this topic is somewhat sparse, but a reasonable summary may be obtained by typing 'bitcoin help' or at https://en.bitcoin.it/wiki/Original_Bitcoin_client/API_calls_list.

ghost commented Feb 12, 2011

May I suggest operating Bitcoin from the command line? The documentation on this topic is somewhat sparse, but a reasonable summary may be obtained by typing 'bitcoin help' or at https://en.bitcoin.it/wiki/Original_Bitcoin_client/API_calls_list.

@sipa

This comment has been minimized.

Show comment Hide comment
@sipa

sipa Feb 21, 2012

Member

@laanwj I read compatibility with NVDA was recently added?

Member

sipa commented Feb 21, 2012

@laanwj I read compatibility with NVDA was recently added?

@laanwj

This comment has been minimized.

Show comment Hide comment
@laanwj

laanwj Feb 22, 2012

Member

Yes; Qt has native support for NVDA, but in windows (with static linking) it was not yet enabled.

I don't know how usable it currently is with screen reader, though. It might be that some hints etc still need to be added.

Member

laanwj commented Feb 22, 2012

Yes; Qt has native support for NVDA, but in windows (with static linking) it was not yet enabled.

I don't know how usable it currently is with screen reader, though. It might be that some hints etc still need to be added.

@gavinandresen

This comment has been minimized.

Show comment Hide comment
@gavinandresen

gavinandresen Apr 4, 2012

Contributor

I'm going to close this, because accessibility support was turned on. If there are still specific issues new issues should be opened.

Contributor

gavinandresen commented Apr 4, 2012

I'm going to close this, because accessibility support was turned on. If there are still specific issues new issues should be opened.

forrestv pushed a commit to forrestv/bitcoin that referenced this issue Sep 12, 2013

Merge pull request #64 from pooler/master-0.8
Add a height parameter to getnetworkhashps

dexX7 added a commit to dexX7/bitcoin that referenced this issue May 30, 2015

Merge pull request #64
7239539 Continue RPC overhaul (input checks, documentation) (dexX7)
3316616 Handle special cases of RPC input data (dexX7)
4612b46 Add class B encoding tests, based on reference data (dexX7)
c0c0066 Add universal unit tests for RequireXYZ() checks (dexX7)
026c778 Add unit tests for ParseXYZ() functions (dexX7)
8b5378b Support generation of coverage data for Omni Core (dexX7)
775048d Run OmniJ RPC tests via script (dexX7)
d630305 Don't print reorganization stats to the console (dexX7)
b83d28e Reduce impact on Bitcoin Core Makefiles (dexX7)
0a21c9a Remove all lcov files, when cleaning up (dexX7)
0c06237 Ignore NetBeans project, lcov and temporary test files (dexX7)

destenson pushed a commit to destenson/bitcoin--bitcoin that referenced this issue Jun 26, 2016

Merge pull request #64 from KevinPayravi/patch-1
Capitalization (these things bug me...)

ptschip pushed a commit to ptschip/bitcoin that referenced this issue Aug 12, 2016

myriadcoin pushed a commit to myriadcoin/myriadcoin that referenced this issue Oct 26, 2017

Merge pull request #64 from cryptapus/myriadcoin.master
housekeeping: rearranged chainparams

CryptAxe pushed a commit to CryptAxe/bitcoin that referenced this issue Mar 11, 2018

Merge pull request #64 from CryptAxe/master
New genesis block with payouts to votecoin addresses, voting with votecoins enabled, voting with bitcoin disabled, RPC fixes, GUI updates.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment