user/dev discussion of public-inbox itself
 help / color / mirror / code / Atom feed
From: Andrei Rybak <rybak.a.v@gmail.com>
To: Eric Wong <e@80x24.org>
Cc: meta@public-inbox.org
Subject: Re: Accesskeys for keyboard navigation
Date: Sun, 15 Jan 2023 12:43:49 +0100	[thread overview]
Message-ID: <c7a0cd68-404a-ea0c-f908-17c699151195@gmail.com> (raw)
In-Reply-To: <20230113005752.M69467@dcvr>

On 13/01/2023 01:57, Eric Wong wrote:
> Andrei Rybak <rybak.a.v@gmail.com> wrote:
>> I'm a frequent user of accesskeys [1] so I've written a user script [2]
>> to add them to some links on websites running public-inbox. The user
>> script is available via Greasy Fork, a website for hosting user scripts:
>>
>>      https://greasyfork.org/en/scripts/458141-accesskeys-for-public-inbox
> 
> Cool.  I've long intended for the HTML output to be stable
> enough something like that.
> 
> Please feel welcome post here if there's new developments or
> discussion to be had about it.
> 
> Fwiw, there's also rel=prev / rel=next for paginated pages;
> but I forgot how browsers use them :x

As far as I know, rel attribute has more to do with search engine
optimization.

However, having these attributes does make finding the links with
document.querySelector much easier for my script.  I've added navigation
for next/prev pages to the script.

> 
>> Any feedback, especially ideas for other uses of accesskey on
>> public-inbox websites would be appreciated.
> 
> Do you know if there's commonly-used accesskeys for normal git repo
> viewers?  (cgit/gitweb/etc...)

None that I know of, and I would guess that it isn't present in any of
them, really.  Confluence (wiki from Atlassian) has them, but Bitbucket
(Git hosting from Atlassian) does not.  (Side note: both of them also
implement their own custom shortcuts).

> The /$INBOX/$OID/s/ endpoint (linked from @@ diff hunk headers)
> is being extended to support "normal" git repositories meant for
> code (WIP: https://80x24.org/lore/pub/scm/git/git.git/ ) and
> I'd like to avoid conflicts.

Conflicts won't be a problem, because there wouldn't be a lot of users
of the user script.  I'll just adjust my script, if needed.

>> P.S. I've looked at source code of public-inbox to try adding support for
>> accesskeys natively, but my Perl knowledge is very limited, so I
>> decided against trying to implement it on my own for a patch.
> 
> No worries.  I will consider it if I find time (some other things
> take priority atm).
> 
> Do you happen to know what happens if a page defines accesskeys
> which conflict with userscripts?  I prefer avoiding breakage for
> existing userscript users if implemented in public-inbox itself.

When there is a conflict, Firefox, as an example, just shows the user
every link with the same accesskey, as if the user was navigating over
them using TAB key.

> I am a bit hesitant to recommend browser extensions in the
> documentation/help output itself since Stylish was found to be
> spyware in the past.  (that goes for downloading and running
> anything from the Internet w/o auditing it, first, public-inbox
> included)

Yeah, the Stylish debacle wasn't pleasant.

      reply	other threads:[~2023-01-15 11:43 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-01-12 23:55 Accesskeys for keyboard navigation Andrei Rybak
2023-01-13  0:57 ` Eric Wong
2023-01-15 11:43   ` Andrei Rybak [this message]

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

  List information: https://public-inbox.org/README

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

  git send-email \
    --in-reply-to=c7a0cd68-404a-ea0c-f908-17c699151195@gmail.com \
    --to=rybak.a.v@gmail.com \
    --cc=e@80x24.org \
    --cc=meta@public-inbox.org \
    /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.
Code repositories for project(s) associated with this public inbox

	https://80x24.org/public-inbox.git

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 read-only IMAP folder(s) and NNTP newsgroup(s).