Dev 0.2 #8
Dev 0.2 #8
Conversation
👷 Deploy Preview for 316john processing.
|
✅ Deploy Preview for 316john ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
src/components/Feedback.jsx
Outdated
| /> | ||
| <input | ||
| type="submit" | ||
| className="p-3 mt-4 bg-zinc-100 dark:bg-widget hover:bg-zinc-200 active:bg-zinc-300 dark:hover:bg-zinc-700 dark:active:bg-zinc-600 w-full font-bold rounded-lg cursor-pointer" |
There was a problem hiding this comment.
тут на ховер и эктив я бы делал затемнение. Мне кажется даже в темной теме это так работает. А у тебя сейчас инверсия. В темной теме на ховер и эктив у тебя наоборот светлее становится. Ну и цвет не совсем тот. Вместо zinc я бы поставил slate или gray. Кстати в темной тебе тоже касается и инпута где пишется язык. Он тоже становится zinc и каким-то светлым на active. Я бы убрал ховер и оставил эктив потемнее.
There was a problem hiding this comment.
Можно поговорить с Владом по поводу этого. Потому что есть ховер на кнопке, инпутах, стрелках и выпадающем вписке. При том что в выпадающем списке там разный дизайн текущего языка и других. Сейчас это не увидеть так как всего один язык.
src/components/VerseSlider.jsx
Outdated
| const VerseSlider = () => { | ||
| let history = createBrowserHistory() | ||
| const { t } = useTranslation() | ||
| const languageGroups = getLanguageGroups() |
There was a problem hiding this comment.
если тебе не нужна сама функция то лучше в хелпере сразу пусть возвращает значение. А то при каждом рендере у тебя будет заново вычисляться это значение
The grouping of verses by language has been added, a link to YouTube has been added, real translations and current texts are used in about the project, goals.