Re: svn commit: r1936515 - subversion/site/staging/docs/release-notes
Daniel Sahlberg <[email protected]> Sat, 25 Jul 2026 17:44:42 +0200
| Newsgroups | gmane.comp.version-control.subversion.devel |
|---|---|
| Message-ID | <CAMHy98MsQ8POW-x2ec2yrNcLZPJYj429G0TPqiOEbFKtAJ6W_g@mail.gmail.com> |
--0000000000001b24cc0657715fcd Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Den fre 24 juli 2026 kl 22:39 skrev Nathan Hartman <[email protected]= m >: > On Thu, Jul 23, 2026 at 6:40=E2=80=AFAM Branko =C4=8Cibej <[email protected]= rg> wrote: > > > > On 23. 7. 2026 12:25, Ivan Zhakov wrote: > > > > On Thu, 23 Jul 2026 at 11:59, <[email protected]> wrote: > >> > >> Author: dsahlberg > >> Date: Thu Jul 23 08:59:43 2026 > >> New Revision: 1936515 > >> > >> Log: > >> In site/staging: > >> > >> * docs/release-notes/1.15.html > >> (#build-system-comparison): Try to add a horizontal scrollbar to the > table > >> instead of resizing the whole page on narrow-width screens. > >> > > [...] > > > >> > >> <details><summary>Build System Comparison</summary> > >> > >> -<table border=3D"1"> > >> +<table border=3D"1" style=3D"display: block; max-width: fit-content; > margin: 0 auto; overflow-x: auto; white-space: nowrap"> > > > > The side effect of this change that table is centered now. All other > tables are left aligned. I think we should left align this table too. > > > > Btw I don't a see problem that table adds scrollbar to all page. > Otherwise we can end up with two scrollbars and this is bad UX in my > opinion. > > > > > > > > I just looked at the staging site, I agree. I also find the arrow > extremely subtle, there's not even a cursor change since it's not a butto= n. > The whole title line should render like a button or a link, otherwise > people will have no idea that there's something to click. > > > > > > -- Brane > > > > I agree the arrow is too subtle and it's not obvious the line can be > clicked to expand/contract. > > Though we could change the text to something like "click to expand" I > feel like having to write that in the text is bad UX. > > So I've experimented with some CSS and this is what I have so far. Not > terribly happy with it but out of time for now... I can try to circle > back later or anyone can feel free to experiment with it further in > the meantime. > > [[[ > > /* > * Make it possible to hide verbose content by default, expanding it > * only on user request > */ > details { > border: 2px solid #4175be; > border-radius: 5px; > > summary { > background-color: #93b0da; > font-size: 115%; > padding: 0.25em; > cursor: pointer; > } > } > > ]]] > > Cheers, > Nathan > I like having the background on the summary element. Not so sure about the blue border. Cheers, Daniel --0000000000001b24cc0657715fcd Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable <div dir=3D"ltr"><div dir=3D"ltr"><span style=3D"background-color:transpare= nt">Den fre 24 juli 2026 kl 22:39 skrev Nathan Hartman <<a href=3D"mailt= o:[email protected]">[email protected]</a>>:</span></div><= div class=3D"gmail_quote gmail_quote_container"><blockquote class=3D"gmail_= quote" style=3D"margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,= 204);padding-left:1ex">On Thu, Jul 23, 2026 at 6:40=E2=80=AFAM Branko =C4= =8Cibej <<a href=3D"mailto:[email protected]" target=3D"_blank">brane@apa= che.org</a>> wrote:<br> ><br> > On 23. 7. 2026 12:25, Ivan Zhakov wrote:<br> ><br> > On Thu, 23 Jul 2026 at 11:59, <<a href=3D"mailto:[email protected]= rg" target=3D"_blank">[email protected]</a>> wrote:<br> >><br> >> Author: dsahlberg<br> >> Date: Thu Jul 23 08:59:43 2026<br> >> New Revision: 1936515<br> >><br> >> Log:<br> >> In site/staging:<br> >><br> >> * docs/release-notes/1.15.html<br> >>=C2=A0 =C2=A0(#build-system-comparison): Try to add a horizontal sc= rollbar to the table<br> >>=C2=A0 =C2=A0 =C2=A0instead of resizing the whole page on narrow-wi= dth screens.<br> >><br> > [...]<br> ><br> >><br> >>=C2=A0 <details><summary>Build System Comparison</su= mmary><br> >><br> >> -<table border=3D"1"><br> >> +<table border=3D"1" style=3D"display: block; ma= x-width: fit-content; margin: 0 auto; overflow-x: auto; white-space: nowrap= "><br> ><br> > The side effect of this change that table is centered now. All other t= ables are left aligned. I think we should left align this table too.<br> ><br> > Btw I don't a see problem that table adds scrollbar to all page. O= therwise we can end up with two scrollbars and this is bad UX in my opinion= .<br> ><br> ><br> ><br> > I just looked at the staging site, I agree. I also find the arrow extr= emely subtle, there's not even a cursor change since it's not a but= ton. The whole title line should render like a button or a link, otherwise = people will have no idea that there's something to click.<br> ><br> ><br> > -- Brane<br> ><br> <br> I agree the arrow is too subtle and it's not obvious the line can be<br= > clicked to expand/contract.<br> <br> Though we could change the text to something like "click to expand&quo= t; I<br> feel like having to write that in the text is bad UX.<br> <br> So I've experimented with some CSS and this is what I have so far. Not<= br> terribly happy with it but out of time for now... I can try to circle<br> back later or anyone can feel free to experiment with it further in<br> the meantime.<br> <br> [[[<br> <br> /*<br> =C2=A0* Make it possible to hide verbose content by default, expanding it<b= r> =C2=A0* only on user request<br> =C2=A0*/<br> details {<br> =C2=A0 border: 2px solid #4175be;<br> =C2=A0 border-radius: 5px;<br> <br> =C2=A0 summary {<br> =C2=A0 =C2=A0 background-color: #93b0da;<br> =C2=A0 =C2=A0 font-size: 115%;<br> =C2=A0 =C2=A0 padding: 0.25em;<br> =C2=A0 =C2=A0 cursor: pointer;<br> =C2=A0 }<br> }<br> <br> ]]]<br> <br> Cheers,<br> Nathan<br></blockquote><div><br></div><div>I like having the background on = the summary element. Not so sure about the blue border.</div><div><br></div= ><div>Cheers,</div><div>Daniel</div><div><br></div></div></div> --0000000000001b24cc0657715fcd--