Skip to content

editor settings, new paragrath and new line, one instance add buttons#4

Merged
dermanov-ru merged 3 commits intomasterfrom
markus3295
Dec 14, 2015
Merged

editor settings, new paragrath and new line, one instance add buttons#4
dermanov-ru merged 3 commits intomasterfrom
markus3295

Conversation

@dermanov-ru
Copy link
Contributor

add new paragrath by pressing enter key
add new line by pressing shift + enter key
add editor settings
make add buttons by one instance and move it through DOM
some css changes

Reviewers

closes #1
closes #2
closes #3
closes #9

add new line by pressing shift + enter key
add editor settings
make add buttons by one instance and move it through DOM
some css changes
@dermanov-ru dermanov-ru changed the title add new paragrath by pressing enter key editor settings, new paragrath and new line, one instance add buttons Dec 9, 2015
ce_interface.js Outdated
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

не вижу смысла сохранять ее в переменную. Мы же потом запишем this.resultTextarea — с ней и можно будет работать. Предлагаю всобачить ее прямо в document.getElementById('codex_editor');

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ну или хотя бы локальной сделать

@eliseealex
Copy link

Мне не нравится, что нельзя вернуться с помощью стрелочек нельзя переходить между блоками.
На странице появляется много лишних плюсов, непонятно, какой к чему относится.

ce_interface.js Outdated

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Как это относится к конфигурации

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Слово Tools не понятно простому смертному, стоит упомянуть, что это виды блоков, которые можно добавить

@dermanov-ru
Copy link
Contributor Author

До сих пор нет удобной навигации между абзацами

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

@eliseealex
Copy link

Я к тому, что перед тем, как двигаться дальше и добавлять новый функционал, надо сделать одну функцию, но очень и очень хорошо.
Я посмотрел задачи, оказывается, там есть задачи на это, так что ладно, удалю прошлый коментарий. То, что я писал будет сделано в рамках #3 и #9.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

3 participants