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

settings/personal not reachable using Screenreader (JAWS/NVDA) #6168

Closed
herrfeuer opened this issue Aug 17, 2017 · 4 comments
Closed

settings/personal not reachable using Screenreader (JAWS/NVDA) #6168

herrfeuer opened this issue Aug 17, 2017 · 4 comments
Labels
bug design Design, UI, UX, etc. feature: accessibility feature: settings good first issue Small tasks with clear documentation about how and in which place you need to fix things in.
Milestone

Comments

@herrfeuer
Copy link

herrfeuer commented Aug 17, 2017

The Settings/Personal Page is not reachable for visually disabled People using Screenreader.
All Objects in Header-Left DIV are missing tags.

Steps to reproduce

  1. Use Jaws or NVDA to navigate on a NC12 Page (apps/files)
  2. Try to reach Settings/Personal Page using the Top Navigation Bar

Expected behaviour

Screenreader tells User which Field is selected by Cursor

Actual behaviour

Screenreader tells User nothing.

Nextcloud version: (see Nextcloud admin page)
12

Updated from an older Nextcloud/ownCloud or fresh install:
fresh

Where did you install Nextcloud from:
git

Signing status:

Signing status
No errors have been found.

</details>

**List of activated apps:**
<details>
<summary>App list</summary>

Enabled:

  • activity: 2.5.2
  • comments: 1.2.0
  • dav: 1.3.0
  • federatedfilesharing: 1.2.0
  • federation: 1.2.0
  • files: 1.7.2
  • files_pdfviewer: 1.1.1
  • files_retention: 1.1.2
  • files_sharing: 1.4.0
  • files_texteditor: 2.4.1
  • files_trashbin: 1.2.0
  • files_versions: 1.5.0
  • files_videoplayer: 1.1.0
  • gallery: 17.0.0
  • logreader: 2.0.0
  • lookup_server_connector: 1.0.0
  • nextcloud_announcements: 1.1
  • notifications: 2.0.0
  • oauth2: 1.0.5
  • password_policy: 1.2.2
  • provisioning_api: 1.2.0
  • serverinfo: 1.2.0
  • sharebymail: 1.2.0
  • systemtags: 1.2.0
  • theming: 1.3.0
  • twofactor_backupcodes: 1.1.1
  • user_saml: 1.3.2
  • workflowengine: 1.2.0
    Disabled:
  • admin_audit
  • encryption
  • files_external
  • firstrunwizard
  • survey_client
  • updatenotification
  • user_external
  • user_ldap
</details>

**Nextcloud configuration:**
<details>
<summary>Config report</summary>

Not relevant.

</details>

**Are you using external storage, if yes which one:** local
**Are you using encryption:** no
**Are you using an external user-backend, if yes which one:** SAML.

### Client configuration
**Browser:**
IE11

**Operating system:**
Windows 10 

### Logs
#### Web server error log
<details>
<summary>Web server error log</summary>

Not relevant.

</details>

#### Nextcloud log (data/nextcloud.log)
<details>
<summary>Nextcloud log</summary>

Not relevant.

</details>

#### Browser log
<details>
<summary>Browser log</summary>

Insert your browser log here, this could for example include:

a) The javascript console log
b) The network log
c) ...

</details>
@MorrisJobke MorrisJobke added design Design, UI, UX, etc. enhancement labels Aug 31, 2017
@MorrisJobke
Copy link
Member

I guess @pixelipo and @jancborchardt looked into this in the past.

@pixelipo
Copy link
Contributor

Reader doesn't read the <span> element in this?

<li data-id="mail" class="">
	<a href="/apps/mail/" tabindex="3">
		<img src="/apps/mail/img/mail.svg" class="app-icon">
		<div class="icon-loading-small-dark" style="display:none;"></div>
	</a>
	<span>Mail</span>
</li>

Then we need to add tittle to <a> - that should solve the problem.

Also related to #5822 - we need to remove deprecated (I think?) .header-appname-container menutoggle element - is that read by screen reader, @herrfeuer ?

We should also figure out whether the Nextcloud logo should actually be a link. That is counter-intuitive, because the <h1> element says "Nextcloud" (screen reader) but the link is to the "Files" app.

@jancborchardt
Copy link
Member

Yeah @herrfeuer, can you give more info on this, how to enhance or fix it? :) Would be much appreciated

@jancborchardt jancborchardt added bug good first issue Small tasks with clear documentation about how and in which place you need to fix things in. and removed enhancement labels Sep 10, 2017
@nextcloud-bot nextcloud-bot added the stale Ticket or PR with no recent activity label Jun 20, 2018
@jancborchardt jancborchardt added this to the Nextcloud 14 milestone Aug 14, 2018
@nextcloud-bot nextcloud-bot removed the stale Ticket or PR with no recent activity label Aug 14, 2018
@jancborchardt
Copy link
Member

This is fixed for Nextcloud 14 with the pull requests #6700 and #9951 (and a lot more improvements on top!) :)

Thank you @herrfeuer for reporting, let us know if there are any further issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug design Design, UI, UX, etc. feature: accessibility feature: settings good first issue Small tasks with clear documentation about how and in which place you need to fix things in.
Projects
None yet
Development

No branches or pull requests

5 participants