Message ID | 20210111120938.20633-1-foss@0leil.net |
---|---|
State | New |
Headers | show |
diff --git a/documentation/sphinx-static/switchers.js b/documentation/sphinx-static/switchers.js index 754de2e6a..fd4dd607d 100644 --- a/documentation/sphinx-static/switchers.js +++ b/documentation/sphinx-static/switchers.js @@ -3,10 +3,10 @@ var all_versions = { 'dev': 'dev (3.3)', - '3.2.1': '3.2.1', - '3.1.4': '3.1.4', - '3.0.4': '3.0.4', - '2.7.4': '2.7.4', + '3.2.1': 'gatesgarth (3.2.1)', + '3.1.4': 'dunfell (3.1.4)', + '3.0.4': 'zeus (3.0.4)', + '2.7.4': 'warrior (2.7.4)', }; var all_doctypes = {
Hi all, Couldn't test properly on a local server so might want to keep an eye on it. As usual, css or js files changes aren't picked up by sphinx so you need to rebuild from scratch. When/If it gets approved, let me know so I can send backport patches. Cheers, Quentin On Mon, Jan 11, 2021 at 01:09:38PM +0100, Quentin Schulz wrote: > Up until now, only the version of a Yocto release was printed, which > isn't very user-friendly. > > Add the release name too so that it's obvious to users which release of > Yocto the currently open docs are applying to. > > Signed-off-by: Quentin Schulz <foss@0leil.net> > --- > documentation/sphinx-static/switchers.js | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/documentation/sphinx-static/switchers.js b/documentation/sphinx-static/switchers.js > index 754de2e6a..fd4dd607d 100644 > --- a/documentation/sphinx-static/switchers.js > +++ b/documentation/sphinx-static/switchers.js > @@ -3,10 +3,10 @@ > > var all_versions = { > 'dev': 'dev (3.3)', > - '3.2.1': '3.2.1', > - '3.1.4': '3.1.4', > - '3.0.4': '3.0.4', > - '2.7.4': '2.7.4', > + '3.2.1': 'gatesgarth (3.2.1)', > + '3.1.4': 'dunfell (3.1.4)', > + '3.0.4': 'zeus (3.0.4)', > + '2.7.4': 'warrior (2.7.4)', > }; > > var all_doctypes = { > -- > 2.29.2 > > > >
> -----Original Message----- > From: docs@lists.yoctoproject.org <docs@lists.yoctoproject.org> On Behalf > Of Quentin Schulz > Sent: den 11 januari 2021 13:12 > To: Quentin Schulz <foss@0leil.net> > Cc: docs@lists.yoctoproject.org > Subject: Re: [docs] [PATCH] documentation: sphinx-static: switchers.js: > add release name in dropdown menu > > Hi all, > > Couldn't test properly on a local server so might want to keep an eye on > it. > > As usual, css or js files changes aren't picked up by sphinx so you need > to rebuild from scratch. > > When/If it gets approved, let me know so I can send backport patches. > > Cheers, > Quentin > > On Mon, Jan 11, 2021 at 01:09:38PM +0100, Quentin Schulz wrote: > > Up until now, only the version of a Yocto release was printed, which > > isn't very user-friendly. > > > > Add the release name too so that it's obvious to users which release of > > Yocto the currently open docs are applying to. > > > > Signed-off-by: Quentin Schulz <foss@0leil.net> > > --- > > documentation/sphinx-static/switchers.js | 8 ++++---- > > 1 file changed, 4 insertions(+), 4 deletions(-) > > > > diff --git a/documentation/sphinx-static/switchers.js b/documentation/sphinx-static/switchers.js > > index 754de2e6a..fd4dd607d 100644 > > --- a/documentation/sphinx-static/switchers.js > > +++ b/documentation/sphinx-static/switchers.js > > @@ -3,10 +3,10 @@ > > > > var all_versions = { > > 'dev': 'dev (3.3)', > > - '3.2.1': '3.2.1', > > - '3.1.4': '3.1.4', > > - '3.0.4': '3.0.4', > > - '2.7.4': '2.7.4', > > + '3.2.1': 'gatesgarth (3.2.1)', > > + '3.1.4': 'dunfell (3.1.4)', > > + '3.0.4': 'zeus (3.0.4)', > > + '2.7.4': 'warrior (2.7.4)', Since these are names, wouldn't it be more appropriate to capitalize them: '3.2.1': 'Gatesgarth (3.2.1)', '3.1.4': 'Dunfell (3.1.4)', '3.0.4': 'Zeus (3.0.4)', '2.7.4': 'Warrior (2.7.4)', > > }; > > > > var all_doctypes = { > > -- > > 2.29.2 //Peter -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#819): https://lists.yoctoproject.org/g/docs/message/819 Mute This Topic: https://lists.yoctoproject.org/mt/79593900/3617530 Group Owner: docs+owner@lists.yoctoproject.org Unsubscribe: https://lists.yoctoproject.org/g/docs/unsub [oe-patchwork@oe-patch.openembedded.org] -=-=-=-=-=-=-=-=-=-=-=-
Hi Peter, On Mon, Jan 11, 2021 at 05:07:49PM +0000, Peter Kjellerstedt wrote: [...] > > On Mon, Jan 11, 2021 at 01:09:38PM +0100, Quentin Schulz wrote: [...] > > > diff --git a/documentation/sphinx-static/switchers.js b/documentation/sphinx-static/switchers.js > > > index 754de2e6a..fd4dd607d 100644 > > > --- a/documentation/sphinx-static/switchers.js > > > +++ b/documentation/sphinx-static/switchers.js > > > @@ -3,10 +3,10 @@ > > > > > > var all_versions = { > > > 'dev': 'dev (3.3)', > > > - '3.2.1': '3.2.1', > > > - '3.1.4': '3.1.4', > > > - '3.0.4': '3.0.4', > > > - '2.7.4': '2.7.4', > > > + '3.2.1': 'gatesgarth (3.2.1)', > > > + '3.1.4': 'dunfell (3.1.4)', > > > + '3.0.4': 'zeus (3.0.4)', > > > + '2.7.4': 'warrior (2.7.4)', > > Since these are names, wouldn't it be more appropriate to capitalize them: > > '3.2.1': 'Gatesgarth (3.2.1)', > '3.1.4': 'Dunfell (3.1.4)', > '3.0.4': 'Zeus (3.0.4)', > '2.7.4': 'Warrior (2.7.4)', > Yup, thanks for the review. In the end, we'll go for a more generic approach so that bitbake and yocto docs can both share switchers.js and use a configuration file to fill the dropdown menu. I'll try to not forget about this review :) We can drop this patch for now. Cheers, Quentin -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#822): https://lists.yoctoproject.org/g/docs/message/822 Mute This Topic: https://lists.yoctoproject.org/mt/79593900/3617530 Group Owner: docs+owner@lists.yoctoproject.org Unsubscribe: https://lists.yoctoproject.org/g/docs/unsub [oe-patchwork@oe-patch.openembedded.org] -=-=-=-=-=-=-=-=-=-=-=-
Up until now, only the version of a Yocto release was printed, which isn't very user-friendly. Add the release name too so that it's obvious to users which release of Yocto the currently open docs are applying to. Signed-off-by: Quentin Schulz <foss@0leil.net> --- documentation/sphinx-static/switchers.js | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-)