Skip to content
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

Баг: если не указать шаблон коммента в треде, нельзя сохранить черновик #1186

Open
2 tasks done
kix opened this issue Mar 14, 2024 · 1 comment

Comments

@kix
Copy link

kix commented Mar 14, 2024

Чеклист

  • Я поискал поиском по трекеру похожие проблемы, в том числе в закрытых Issues
  • Баг стабильно воспроизводится и я знаю как это сделать

Описание бага

Пишу тред, не заполняю поле под шаблон, пытаюсь сохранить черновик, но нажание на кнопку ни к чему не приводит.

В консоли браузера вываливается:

An invalid form control with name='comment_template' is not focusable. 
<textarea name=​"comment_template" cols=​"40" rows=​"10" maxlength=​"5000" placeholder=​"Здесь тоже поддерживается и рекомендуется Markdown" required=​"required" id=​"id_comment_template" class=​"markdown-editor-full" style=​"display:​ none;​">​</textarea>​

Ожидаемый результат

Черновик должен сохраниться, даже если я не указал шаблон. Шаблон должен быть обязательно заполнен только при публикации

Шаги к воспроизведению

  1. Зайти на https://vas3k.club/create/thread/
  2. Заполнить заголовок и текст треда
  3. Выбрать «публичный пост» или «только для своих»
  4. Нажать «Сохранить черновик»
@trin4ik
Copy link
Contributor

trin4ik commented Jun 15, 2024

Идея в тои, что easymde делает display: none на текстарею, которая required. При попытке сохранения черновика, нет никаких сообщений о ошибке явных, только в консоль срётся факт того, что нельзя поставить фокус на элемент, который не отображён.
Посмотрел easymde, всё что нашёл -- ишью от 19 года и признеание факта, что required никак не обрабатывается
Ionaru/easy-markdown-editor#105

Пути решения:

  • вместо disaplay: none делать position: absolute;left:0;opacity: 0; display: block !important, т.е. можно прям поверх накатить
  • убрать required=True для предпросмотра/черновика.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants