New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Unsupported browsers #206
Unsupported browsers #206
Conversation
frontstall
commented
Aug 18, 2021
app/helpers/packs_helper.rb
Outdated
@@ -13,7 +13,9 @@ def prepend_javascript_packs(*packs) | |||
end | |||
|
|||
def render_javascript_stored_packs | |||
javascript_pack_tag(*@javascript_packs, crossorigin: 'anonymous') | |||
if supported_browser? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Эта логика должна быть снаружи, а не внутри. Уровни абстракции все дела
@@ -0,0 +1 @@ | |||
.alert.alert-warning[role="alert"] = t('.error') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Стандарт именования не тот, это не ошибка, это объяснение. Нужно исходить из того, что ключ на английском без перевода будет смотреться смотрибельно
@@ -4,6 +4,8 @@ html.x-h-md-100 lang="#{I18n.locale}" | |||
= render 'layouts/shared/head' | |||
= Gon::Base.render_data | |||
body.d-flex.flex-column.x-h-md-100 | |||
- if !supported_browser? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- unless ...
config/locales/ru.layouts.yml
Outdated
@@ -26,3 +26,5 @@ ru: | |||
recommended_books: Рекомендуемые книги | |||
matrix: Матрица компетентности | |||
source_code: Исходный код | |||
unsupported_browser_warning: | |||
error: Вы используете устаревший браузер. Некоторые возможности отключены. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Формулировка агрессивная, на людей низя указывать. "Ваш браузер устарел". Нужно всегда уходить от указаний на людей
2c3df6e
to
6c23424
Compare