Skip to content

Commit

Permalink
Add erb-lint with github accessibility rules
Browse files Browse the repository at this point in the history
Removed title from a tag to satisfy: The title attribute should never be used for an a tag

Co-authored-by: Jane Sandberg <sandbergja@users.noreply.github.com>
Co-authored-by: Liz Garcia <lizgarciao@users.noreply.github.com>
  • Loading branch information
3 people committed Sep 5, 2023
1 parent b638fc0 commit 6b5c949
Show file tree
Hide file tree
Showing 5 changed files with 27 additions and 1 deletion.
5 changes: 5 additions & 0 deletions .erb-lint.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
EnableDefaultLinters: false
inherit_gem:
erblint-github:
- config/accessibility.yml
1 change: 1 addition & 0 deletions .erb-linters/erblint-github.rb
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
require "erblint-github/linters"
2 changes: 2 additions & 0 deletions Gemfile
Original file line number Diff line number Diff line change
Expand Up @@ -118,6 +118,8 @@ group :development, :test do
gem 'bixby', '~> 5.0'
gem 'capybara'
gem 'coveralls_reborn'
gem "erb_lint", require: false
gem "erblint-github"
gem 'pry-byebug'
gem 'solargraph'
end
18 changes: 18 additions & 0 deletions Gemfile.lock
Original file line number Diff line number Diff line change
Expand Up @@ -108,6 +108,13 @@ GEM
bcrypt (3.1.18)
bcrypt_pbkdf (1.1.0)
benchmark (0.2.1)
better_html (2.0.2)
actionview (>= 6.0)
activesupport (>= 6.0)
ast (~> 2.0)
erubi (~> 1.4)
parser (>= 2.4)
smart_properties
bixby (5.0.2)
rubocop (= 1.28.2)
rubocop-ast
Expand Down Expand Up @@ -226,6 +233,14 @@ GEM
ed25519 (1.3.0)
email_validator (2.2.4)
activemodel
erb_lint (0.5.0)
activesupport
better_html (>= 2.0.1)
parser (>= 2.7.1.4)
rainbow
rubocop
smart_properties
erblint-github (0.4.1)
erubi (1.12.0)
execjs (2.8.1)
ezwadl (0.1.0)
Expand Down Expand Up @@ -523,6 +538,7 @@ GEM
simplecov-html (0.12.3)
simplecov_json_formatter (0.1.4)
slop (4.10.1)
smart_properties (1.17.0)
sneakers (2.11.0)
bunny (~> 2.12)
concurrent-ruby (~> 1.0)
Expand Down Expand Up @@ -667,6 +683,8 @@ DEPENDENCIES
devise-guests (~> 0.5)
ed25519
email_validator
erb_lint
erblint-github
factory_bot_rails
faker
faraday (~> 0.17)
Expand Down
2 changes: 1 addition & 1 deletion app/views/shared/_header_navbar.html.erb
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
<div class="container">
<div class="row align-items-center">
<div class="logo__pul col-12 col-sm-auto">
<a href="https://library.princeton.edu" title="Princeton University Library">
<a href="https://library.princeton.edu">
<%= image_tag("pul-logo-new.svg", alt: "Link to Princeton University Library home page") %>
</a>
<%=link_to t('blacklight.orangelight').html_safe, main_app.root_path, :class => "site-title" %>
Expand Down

0 comments on commit 6b5c949

Please sign in to comment.