tech@mandoc.bsd.lv
 help / color / mirror / Atom feed
From: Ingo Schwarze <schwarze@usta.de>
To: Anna <cyber@sysrq.in>
Cc: tech@mandoc.bsd.lv
Subject: Re: [PATCH 0/8] Make generated HTML more accessible
Date: Wed, 22 Jun 2022 21:15:33 +0200	[thread overview]
Message-ID: <YrNqVTDpBr+7XcLI@asta-kit.de> (raw)
In-Reply-To: <20220621122749.11417-1-cyber@sysrq.in>

Hello Anna,

thank you for sending your patches.
I agree that accessibility is an important goal.

Anna wrote on Tue, Jun 21, 2022 at 05:27:41PM +0500:

> For reference:
> * https://www.w3.org/TR/dpub-aria-1.1/
> * https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/

These documents are extremely long (which is unfortunately the case
for many other W3C documents too, and which is a common effect of
design by committee in general).  An unfortunate consequence is that
i will have to read them selectively, and even that may take time.

Three other goals of the mandoc project, in addition to accessibility,
are (1) simplicity, (2) standard conformance, and (3) consistency.
There are more goals, like correctness, robustness, security etc., but
those matter less in the present context.

These goals have a number fo implications:

 - (1) simplicity means that i do not want to add redundant
       or unnecessarily long markup.  For example, i would not
       want to add ARIA properties if the HTML element used
       already implies the role, or can be changed to imply the
       role.  As another example, when adding a new wrapper
       element for ARIA purposes, i would want to check whether
       the immediately surrounding, adjacent, or contained
       elements can be simplified.  From a brief review, i
       suspect that your patches are already quite good in this
       respect, but i want to check such aspects for every change.

 - (2) standard conformance is in particular related to
       https://www.w3.org/TR/wai-aria-1.1/ section 1.4
       Co-Evolution.  I want both HTML elements and properties
       and ARIA properties used as standardized, but i fear
       there may still be places where elements are abused
       and more standard-conforming elements can be chosen.
       So whenever changing an element, i want to check the
       immediate vicinity for conformance.

 - (3) consistency implies that if one technique is chosen -
       respecting simplicity and conformance as described above -
       at one place, then it should be used throughout for the
       respective mdoc(7) or man(7) macro or for the respective
       HTML element.  This is particularly important for
       languages that are as large as HTML, CSS, and ARIA:
       They usually provide several different ways to achieve
       a similar effect, and unless consistency is taken very
       seriously, markup chaos typically ensues.

For these reasons, i never accept patch series.

Instead, i will consider each patch separately.
Let's see how that goes.

One final question:
Do you have a good way for testing?
Is there a difference in the diff in the amount of effort for:
1. testing an uncommitted diff
2. testing a committed diff
3. testing a diff deployed to man.openbsd.org ?

Yours,
  Ingo
--
 To unsubscribe send an email to tech+unsubscribe@mandoc.bsd.lv


  parent reply	other threads:[~2022-06-22 19:15 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-06-21 12:27 Anna “CyberTailor”
2022-06-21 12:27 ` [PATCH 1/8] mdoc_html: Accessibility markup for ToC Anna “CyberTailor”
2022-06-21 12:27 ` [PATCH 2/8] mdoc_html: Add DPUB-ARIA roles to sections Anna “CyberTailor”
2022-06-24 13:58   ` Ingo Schwarze
2022-06-21 12:27 ` [PATCH 3/8] mdoc_html: Add DPUB-ARIA roles to subsections Anna “CyberTailor”
2022-06-21 12:27 ` [PATCH 4/8] man_html: Add DPUB-ARIA roles to (sub)sections Anna “CyberTailor”
2022-06-21 12:27 ` [PATCH 5/8] man.cgi: add ARIA roles and semantics Anna “CyberTailor”
2022-07-04 16:37   ` Ingo Schwarze
2022-06-21 12:27 ` [PATCH 6/8] mdoc_html: Add accessible description to crosslinks Anna “CyberTailor”
2022-06-25 12:58   ` Ingo Schwarze
2022-06-21 12:27 ` [PATCH 7/8] mdoc_html: Tell screen readers to skip the header Anna “CyberTailor”
2022-06-26 15:51   ` Ingo Schwarze
2022-06-28 16:33     ` Anna “CyberTailor”
2022-06-21 12:27 ` [PATCH 8/8] man_html: " Anna “CyberTailor”
2022-06-22 19:15 ` Ingo Schwarze [this message]
2022-06-22 19:40   ` [PATCH 0/8] Make generated HTML more accessible Anna “CyberTailor”
2022-06-23 13:34     ` Ingo Schwarze

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=YrNqVTDpBr+7XcLI@asta-kit.de \
    --to=schwarze@usta.de \
    --cc=cyber@sysrq.in \
    --cc=tech@mandoc.bsd.lv \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).