On Tue, 6 May 2025 at 11:34, sebb <seb...@gmail.com> wrote: > > On Tue, 6 May 2025 at 10:32, Niall Pemberton <niall.pember...@gmail.com> > wrote: > > > > On Tue, 6 May 2025 at 09:54, sebb <seb...@gmail.com> wrote: > > > > > On Tue, 6 May 2025 at 09:25, Niall Pemberton <niall.pember...@gmail.com> > > > wrote: > > > > > > > > On Tue, 6 May 2025 at 09:23, Niall Pemberton <niall.pember...@gmail.com> > > > > wrote: > > > > > > > > > On Tue, 6 May 2025 at 01:25, sebb <seb...@gmail.com> wrote: > > > > > > > > > >> https://attic-acevent.staged.apache.org/ > > > > >> shows a possible layout to keep the logo next to the Events link > > > > >> > > > > >> Code is acevent branch. > > > > >> The solution is somewhat crude, but it does seem to work. > > > > >> > > > > > > > > > > Thanks Sebb, that looks better - +1 to this change > > > > > > > > > > I also tried moving the "Improve this page" to the "Breadcrumbs" - we > > > lose > > > > > the "breadcrumbs" functionality, by overriding the "include" - but > > > > > seem > > > > > like a better option to me > > > > > > > > > > WDYT? > > > > > > > > > > > > > Meant to say - tried it in the same branch > > > > https://attic-acevent.staged.apache.org/ > > > > > > > > > > Not sure we need the breadcrumbs as the site is not deeply hierarchical. > > > > > > The View/Improve buttons are more useful in the new place, but I'm not > > > sure we need both. > > > Also 'View this page' seems a bit odd as a title - aren't we already > > > viewing the page? > > > It might be better as 'View page source', if we decide to keep it as > > > well as the edit link. > > > > > > > Good points - happy to EITHER just have the "Improve this page" link OR > > re-word the "View this page" as you suggest. > > OK, done.
Note: I think the View page source link is needed. The edit link does not show the contents of the file if the user does not have write access, and the display does not include a link to the source file. > > > Niall > > > > > > > > > > > > > > > > > > Niall > > > > > > > > > > > > > > >> > > > > >> Sebb > > > > >> > > > > > > > >
