glokascom / bibleVis

https://biblevis.com
0 stars 0 forks source link

Feature pavel 33 #92

Closed Pavel064 closed 2 months ago

Pavel064 commented 2 months ago

Работа нуждается в рефакторинге:

netlify[bot] commented 2 months ago

Deploy Preview for biblevis ready!

Name Link
Latest commit a795ac856078b7bbc19d84c3e09723d07f38c126
Latest deploy log https://app.netlify.com/sites/biblevis/deploys/66d1b6b5da051f0008ec3b43
Deploy Preview https://deploy-preview-92--biblevis.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

foxprogs commented 2 months ago

изображение

  1. Если нажимают replace image и грузят картинку которая выкидывает ошибку, то сейчас перекидывает на начальную страницу. Так не надо, если открылась вот эта страница, где ты заполняешь что-то, с нее не надо уходить. Просто пусть появится ошибка где-то, что не тот фортмат или размер. И пусть остается предыдущая картинка.
  2. Если заполнены все обязательные поля то кнопка должна быть зеленая. Это primary кнопка. Сейчас она серая но активная. Я нашел в чем была проблема, я там в классе написал background-secondary-50 а надо bg-secondary-50
  3. Если это не AI generation то надо скрывать только промпт, а софт остается. Это может быть адоб фотошоп, прокриэйт и т.д. @Pavel064
foxprogs commented 2 months ago

изображение теги работают как-то не так. Я выбираю его, и он все равно в списке остается. И область очень маленькая, надо чтобы 2 строки помещалось

foxprogs commented 2 months ago

изображение Если не заполнено ни одного поля то кнопка isDisabled={true}

И большой вопрос, а мы позволяем менять картинку после ее публикации? Мне кажется надо запретить. @Pavel064 @Valyukhov