Добавил поддержку параметров для эмоджи

[rvrhiv]

No description provided.

[rvrhiv]

@sergeche нужно ли тут проверить, что в emoji: string содержится только эмоджи без других символов?
Я тестировал, и этот метод отрабатывает нормально, когда передаем просто текст (например "123"), текст + эмоджи ("123😀"), несколько эмоджи ("😀😀", тогда параметры применятся ко всем переданным эмоджи)

[rvrhiv]

• Нужно ли добавить тесты для этой фичи?
• Нужно ли дописать README.md?

[sergeche]

В целом получилось несколько разных методов, которые делают одни и те же изменения, но в разных местах и с разным контрактом. Хотя по сути всё сводится к одной простой задаче: для эмоджи, который находится в позиции pos, нужно указать объект с параметрами. Я бы сделал одну функцию на решение этой задачи. Что-то типа такого:

interface EmojiUpdatePayload {
	/** Позиция эмоджи относительно всей строки */
	pos: number;
	/** Данные, которые нужно добавить. Если `null` — удалить данные */
	data: EmojiData | null;
	/** 
	 * Эмоджи-подсказка. Если указано, то сначала проверим, что эмоджи в модели
	 * совпадает с переданным. Если не совпадает — ничего не меняем
	 */
	hint?: string;
}

function updateEmojiData(tokens: Token[], payload: EmojiUpdatePayload[]): Token[] {
	// 1. Убедимся, что payloads отсортирован
	// 2. Для каждого элемент из payload находим токен и эмоджи в нём
	// 3. Если есть hint, убедимся, что эмоджи совпадают
	// 4. Если всё нашли и надо менять — обновляем токен
}

И в API редактора не делать отдельный метод для вставки эмоджи, а делать метод именно для изменения данных в эмоджи. И ты на стороне UI будешь выполнять две операции: вставлять/заменять текст и добавлять кастомные данные для эмоджи

[sergeche]

Да, теперь похоже на правду. Вроде всё хорошо

[sergeche]

@tamtam-chat/message-editor@0.10.0