Re: [PATCH RFC V2] docs: Add more information to the HTML sidebar

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Date: Sat, 4 Feb 2023 09:20:13 +0900, Akira Yokosawa wrote:
> On Fri, 03 Feb 2023 17:02:17 -0700, Jonathan Corbet wrote:
>> Jonathan Corbet <corbet@xxxxxxx> writes:
>>
>>> As before, the results can be seen at:
>>>
>>>   https://static.lwn.net/kerneldoc/
>>
>> What's there now reflects a couple of tweaks beyond the patch I sent.
>>
>> Hopefully I've addressed some of the concerns...  One thing I really
>> don't like about the auto-positioning sidebar, though, is that it pushes
>> the "The Linux Kernel" heading off-screen.  For somebody following a
>> link into the docs from elsewhere, that could be a bit confusing.
> 
> That's the same in RTD. So this might be a point you hate RTD, I guess.
> 
> But I see a major problem on small/narrow screens.
> The sidebar is now kept at the top, and by clicking/tapping a link in the
> TOC, I can't jump to the section I want to see.
> 
> Sorry, but this is not usable at all.

To be clear, I meant "on small/narrow screens".
On normal PC screens, this resolves the issues I mentioned earlier.

So I'm OK with this goes into v6.3. Maybe add a note on small screen
devices in the Changelog.

BTW, I very much like the way LWN's site navigation behaves on small
screen devices. LWN is designed to be accessible without hierarchical
structure of documents, which is the opposite of how kernel documentation
is managed based on the nested TOC entries.

        Thanks, Akira

> 
>>                                                                    I
>> know vaguely how to fix it with CSS, but getting such things right
>> always require a fair amount of dinking around and I can't do that now.
> 
> ;-) ;-) ;-)
> 
>         Thanks, Akira
> 
>>
>> jon



[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux FS]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux