hexlet-rus / runit

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

fixed html extension when making new snippet #470

Closed bobrov-site closed 5 months ago

bobrov-site commented 5 months ago

469 решение проблемы со расширением сниппета при его создании

fey commented 5 months ago

@bobrov-site давайте уберем расширение. Имя сниппета может любым и может содержать расширение, а может не содержать. Если у нас будет сниппет с несколькими файлами, то как указать имя сниппета? ПРоще не подтягивать расширение в имя сниппета. Для обозначения языка, на котором сниппет, у нас есть отдельный атрибут.

bobrov-site commented 5 months ago

@fey хорошо. Займусь этим сегодня вечером или уже завтра

bobrov-site commented 5 months ago

@fey у меня получилось исправить так, чтобы новые сниппеты создавались без расширения, но у меня возник вопрос, что делать со старыми у которых есть расширение в названии? Опять же, расширение = название в данном случае.

Есть конечно вариант создать какую-то функцию обертку, которая будет скрывать все, что написано после последней точки, но нужно будет повозиться.

Как быть?)

bobrov-site commented 5 months ago

в snippetData, name это то, что содержит имя+расширение у старых сниппетов https://github.com/hexlet-rus/runit/blob/5f5a8866806146c9230d2cd998e1857afed66ee7/frontend/src/pages/snippet/FileToolbar.jsx#L25

fey commented 5 months ago

@bobrov-site просто оставляем как есть. название используется только для отображения