bespoyasov / solidbook

Book about the SOLID principles and object-oriented software design.
https://solidbook.vercel.app
MIT License
602 stars 59 forks source link

Improve accessability for footer external links #125

Closed gebeto closed 5 years ago

gebeto commented 5 years ago

Opens links to external site in new window by default

vercel[bot] commented 5 years ago

This pull request is automatically deployed with Now. To access deployments, click Details below or on the icon next to each push.

Latest deployment for this branch: https://ota-solid-git-fork-gebeto-master.ota.now.sh

dex157 commented 5 years ago

Eslint-a11y вроде бы просит кроме таргета ещё какие то атрибуты тэгов указывать для external ссылок

gebeto commented 5 years ago

Возможно rel="noopener" Вписал еще его

bespoyasov commented 5 years ago

А почему мы хотим открывать эти ссылки в новом окне?

gebeto commented 5 years ago

Так удобнее, что б не уходить с нашего сайта для просмотра профилей авторов ● открыл профиль -> полазил по репозиториям/почитал об авторах -> закрыл вкладку

сейчас же: ● открыл профиль -> полазил по репозиториям/почитал об авторах -> НАЗАД -> НАЗАД -> НАЗАД -> НАЗАД -> НАЗАД

✚ если pwa сохранить в офлайн и в офлайне запускать то будет просто ошибка соединения, с _blank будет открыватся браузер и хоть будет url видно

У вас там кстати pwa настроено, манифест есть, но next-offline не подключен и не работает оффлайн на айфоне(на андроиде не тестил), хотя может вы и не планировали pwa делать, но если пару строчек добавить то оно заработает

dex157 commented 5 years ago

@gebeto, pwa в планах было, руки не дошли его сделать. @bespoyasov я согласен с тем, что на external ресурсы нужно ставить target=_blank ссылки. Правда, это на уровне ощущения, чёт рационализировать для себя такое поведение я не смог =)

gebeto commented 5 years ago

@dex157 я кстати может что то не так делал, но с target: serverless pwa оффлайн мне завести не удалось, только с target: server > next build > next export

хотя возможно это изза того что я в now dev на локалке запускал

bespoyasov commented 5 years ago

Ну у меня политика в отношении ссылок — «дать пользователям самим решать, в какой вкладке открыть страницу».

Но вот это:

если pwa сохранить в офлайн и в офлайне запускать то будет просто ошибка соединения, с _blank будет открыватся браузер и хоть будет url видно

меня убедило :–)