hexlet-rus / runit

Online Editor Form Embedding
https://runit.hexlet.ru/
Other
35 stars 121 forks source link

Создать кнопку Сохранить на страницу сниппета #441

Closed Yawerica closed 4 months ago

Yawerica commented 7 months ago

Problem description

Отсутствует кнопка Закрыть в открытом сниппете

Proposed solution

Создать кнопку Закрыть. Кнопка несет чисто психологический характер) возможно лучше не просто Закрыть, а Сохранить и Закрыть

Solution

Добавляем кнопку "Сохранить" для сохранения сниппета вручную.

добавить кнопку закрыть

fey commented 7 months ago

Тут такой момент - страница каждого сниппета - это именно страница, а не окошко на какой-то другой странице. Поэтому концепция "закрыть", тут не работает - непонятно, что показать.

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

bobrov-site commented 6 months ago

Я могу попробовать взять в работу, если ишью актуальное

bobrov-site commented 6 months ago

У меня возник по работе кнопки. Как ее нужно сделать в конечном счете? Я изначально подумал, что если нажать на кнопку сохранить, то сохраняется код (просто вызываем принудительное сохранение) и нас перебрасывает в мои снипеты.

Просто если кнопка будет работать, как просто сохранение - то смысла в ней нет, так как код итак сохраняется, когда мы создаем файл или же просто что-то написали

bobrov-site commented 6 months ago

Добавил пулреквест, дайте пожалуйста уточнение по работе кнопки. Если что подправлю

Yawerica commented 6 months ago

Считаю, что для пользователя неочевидно, что код сохраняется автоматически. Облачко со словом Сохранено не достаточно убедительно. Может его как то выделить? или я не права?)

fey commented 6 months ago

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

fey commented 6 months ago

Просто если кнопка будет работать, как просто сохранение - то смысла в ней нет, так как код итак сохраняется, когда мы создаем файл или же просто что-то написали

кнопка должна сохранять сниппет. Вполне может быть такое. что автосохранение не сработает. Если вам нужно будет сохранить сниппет, то придется его изменять, внося ненужные изменения. А по кнопке вы можете сохранить его, не меняя.

faciledictu commented 5 months ago

Совершенно не согласен с тем, что нужна кнопка "Сохранить". Она только путает пользователя и усложняет интерфейс, не предлагая никакой функциональности. Сохранение происходит автоматически. Надпись о том, что код сохранен – куда же убедительнее? :) Это фактический стандарт работы с облачными редакторами. Давайте посмотрим, как реализовано сохранение в облачных сервисах

Google Drive Undsaved Google Drive Saved Screenshot 2024-04-26 at 14 13 25
fey commented 5 months ago

Обкашляем с Кириллом

faciledictu commented 5 months ago

А вот нашел пример очень путающего паттерна. Смотри, я добавил в открытый строку с H2 прямо сейчас. И на вкладке с открытым файлом вместо крестика появилась жирная белая точка (типа файл не сохранен). Но он по факту сохранен. Я уже сейчас вижу изменение верстки в панели справа. Плюс я тупо закрыл браузер, открыл его, и в файле все изменения отображаются. Все равно он его сохранял (несмотря на точку). Его невозможно закрыть, не сохраняя изменения.

Screenshot 2024-04-26 at 14 30 25
bobrov-site commented 4 months ago

@fey есть информация по кнопке?

fey commented 4 months ago

еще не созванивался. Пока кнопку оставляем, убрать всегда можем.

Yawerica commented 4 months ago

@fey возможно мы правда погорячились с кнопкой Сохранить. потому что ЦА RUN IT это программисты и скорее всего такие люди знакомы с Интерфейсом облачных сервисов как говорит @faciledictu. и в курсе что Облачко в центре означает что код Сохранен. НО предлагаю кнопку ОЧИСТИТЬ или Иконку корзины вместо созданной кнопки Сохранить. чтобы очистить консоль. Как в Replit например) кнока очистить извините за созданную смуту с кнопкой.

fey commented 4 months ago

С кнопкой пока не переживайте. Пусть будет. А насчет очистить вывод - можно добавить, да. Создайте ищщус.