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.

> Niall
>
>
> > >
> > > >
> > > > Niall
> > > >
> > > >
> > > >>
> > > >> Sebb
> > > >>
> > > >
> >

Reply via email to