search

bokeh / bokeh.org

Static top-level "brand" page
BSD 3-Clause "New" or "Revised" License
6 stars 14 forks source link

Missing HTML lang attribute from home page #23

Closed chinmaychahar closed 1 year ago

chinmaychahar commented 1 year ago

Software versions

Python version : 3.9.13 (main, Aug 25 2022, 23:51:50) [MSC v.1916 64 bit (AMD64)] IPython version : 7.31.1 Tornado version : 6.1 Bokeh version : 2.4.3 BokehJS static path : C:\Users\Chinmay\anaconda3\lib\site-packages\bokeh\server\static node.js version : v16.17.0 npm version : 8.19.2 Operating system : Windows-10-10.0.19044-SP0

Browser name and version

Brave Version 1.48.171

Jupyter notebook / Jupyter Lab version

No response

Expected behavior

Since I was going through the documentation regarding #7, one small thing I noticed was that the home page (https://bokeh.org/) missed the HTML lang attribute which is helpful for people using screen readers and comes under many web accessibility guidelines (ref).

It should be set to en here just like it's set for gallery page

Observed behavior

lang attribute missing from home page

Example code

`<html>` to `<html lang="en" ..>`

Stack traceback or browser console output

No response

Screenshots

Home Page- bokeh_page Gallery Page- gallery_page

chinmaychahar commented 1 year ago

I realize this is a small task but I couldn't find any existing relevant issues for the same. I can work on this cc @bryevdv

bryevdv commented 1 year ago

Hi @chinmaychahar You are correct that would be good to add. Note I have transferred this issue to a different repo https://github.com/bokeh/bokeh.org The "top level" site is separate from the docs in the main repo.

bryevdv commented 1 year ago

I think that updating the templates in this directory might be sufficient:

https://github.com/bokeh/bokeh.org/tree/main/_layouts

Note that that static https://bokeh.org site was very hastily put together by someone who does not have much website building experience (me) so there are probably lots of improvements that can be made. Any suggestions you have are very welcome!

chinmaychahar commented 1 year ago

Okay, thanks. I'll start working on this asap!

chinmaychahar commented 1 year ago

@bryevdv should I update the website home page screengrab in the README.md too? The current one seems outdated

bryevdv commented 1 year ago

@chinmaychahar good eye! yes that would certainly be reasonable to update since the front page has changed

chinmaychahar commented 1 year ago

I'm trying to follow the instructions given in the readme and run the site locally using docker. 

C:\Users\Chinmay>docker --version
Docker version 20.10.23, build 7155243

However, I'm getting the following error while running make serve

PS C:\Users\Chinmay\bokeh.org> make serve
Launching dockerized site at http://localhost:4000
docker run --rm -v /cygdrive/c/Users/Chinmay/bokeh.org:/srv/jekyll -it -p 4000:4000 jekyll/jekyll:pages jekyll serve --incremental
ruby 3.1.1p18 (2022-02-18 revision 53f5fc4236) [x86_64-linux-musl]
Configuration file: none
            Source: /srv/jekyll
       Destination: /srv/jekyll/_site
 Incremental build: enabled
      Generating...
jekyll 3.9.2 | Error:  Permission denied @ dir_s_mkdir - /srv/jekyll/_site
/usr/local/lib/ruby/3.1.0/fileutils.rb:243:in `mkdir': Permission denied @ dir_s_mkdir - /srv/jekyll/_site (Errno::EACCES)
        from /usr/local/lib/ruby/3.1.0/fileutils.rb:243:in `fu_mkdir'
        from /usr/local/lib/ruby/3.1.0/fileutils.rb:221:in `block (2 levels) in mkdir_p'
        from /usr/local/lib/ruby/3.1.0/fileutils.rb:219:in `reverse_each'
        from /usr/local/lib/ruby/3.1.0/fileutils.rb:219:in `block in mkdir_p'
        from /usr/local/lib/ruby/3.1.0/fileutils.rb:211:in `each'
        from /usr/local/lib/ruby/3.1.0/fileutils.rb:211:in `mkdir_p'
        from /usr/gem/gems/jekyll-3.9.2/lib/jekyll/convertible.rb:226:in `write'
        from /usr/gem/gems/jekyll-3.9.2/lib/jekyll/site.rb:209:in `block in write'
        from /usr/gem/gems/jekyll-3.9.2/lib/jekyll/site.rb:332:in `block (2 levels) in each_site_file'
        from /usr/gem/gems/jekyll-3.9.2/lib/jekyll/site.rb:331:in `each'
        from /usr/gem/gems/jekyll-3.9.2/lib/jekyll/site.rb:331:in `block in each_site_file'
        from /usr/gem/gems/jekyll-3.9.2/lib/jekyll/site.rb:330:in `each'
        from /usr/gem/gems/jekyll-3.9.2/lib/jekyll/site.rb:330:in `each_site_file'
        from /usr/gem/gems/jekyll-3.9.2/lib/jekyll/site.rb:208:in `write'
        from /usr/gem/gems/jekyll-3.9.2/lib/jekyll/site.rb:73:in `process'
        from /usr/gem/gems/jekyll-3.9.2/lib/jekyll/command.rb:28:in `process_site'
        from /usr/gem/gems/jekyll-3.9.2/lib/jekyll/commands/build.rb:65:in `build'
        from /usr/gem/gems/jekyll-3.9.2/lib/jekyll/commands/build.rb:36:in `process'
        from /usr/gem/gems/jekyll-3.9.2/lib/jekyll/commands/serve.rb:93:in `block in start'
        from /usr/gem/gems/jekyll-3.9.2/lib/jekyll/commands/serve.rb:93:in `each'
        from /usr/gem/gems/jekyll-3.9.2/lib/jekyll/commands/serve.rb:93:in `start'
        from /usr/gem/gems/jekyll-3.9.2/lib/jekyll/commands/serve.rb:75:in `block (2 levels) in init_with_program'
        from /usr/gem/gems/mercenary-0.3.6/lib/mercenary/command.rb:220:in `block in execute'
        from /usr/gem/gems/mercenary-0.3.6/lib/mercenary/command.rb:220:in `each'
        from /usr/gem/gems/mercenary-0.3.6/lib/mercenary/command.rb:220:in `execute'
        from /usr/gem/gems/mercenary-0.3.6/lib/mercenary/program.rb:42:in `go'
        from /usr/gem/gems/mercenary-0.3.6/lib/mercenary.rb:19:in `program'
        from /usr/gem/gems/jekyll-3.9.2/exe/jekyll:15:in `<top (required)>'
        from /usr/gem/bin/jekyll:25:in `load'
        from /usr/gem/bin/jekyll:25:in `<main>'
make: *** [Makefile:3: serve] Error 1

I tried to change the read/write permissions as well as manually creating the _site directory but nothing works. Any suggestions on what might be wrong? @bryevdv

bryevdv commented 1 year ago

@chinmaychahar I'll have to try to take a closer look soon. I haven't need or tried to build this site in a few years at least (and never on windows) 😓

chinmaychahar commented 1 year ago

Oh okay, I wonder if it might be due to some outdated requirements then. I'll try my best to see if I can solve it @bryevdv

bryevdv commented 1 year ago

👍 definitely we can update the README instructions too if that's the case

chinmaychahar commented 1 year ago

Yes, okay