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

feat: save and restore grouped tabs selection #78

Merged
merged 13 commits into from
Mar 4, 2025

Conversation

bazuka5801
Copy link
Contributor

@bazuka5801 bazuka5801 commented Feb 20, 2025

depends on diplodoc-platform/client#105

Изменения

  • Добавлена новая функциональность для сохранения состояния выбора сгруппированных вкладок.
  • Реализовано механизм восстановления выбора вкладок при загрузке приложения.
  • Обновлены соответствующие тесты для проверки новой функциональности.

Тестирование

Для убедительного тестирования произведенного функционала:

  1. Выполните группировку нескольких вкладок в приложении.
  2. Выберите нужные вкладки в группе.
  3. Перезагрузите страницу или повторно откройте приложение.
  4. Убедитесь, что выбранные ранее вкладки сохраняют свое состояние.

Пример группированных табов

{% list tabs group=instructions %} 
 
- Java 
 
  Текст таба 1. 
 
  * Можно использовать списки. 
  * И другую разметку. 
 
- Go 
 
  Текст таба 2. 
 
- C-sharp 
 
  Текст таба 4. 
 
{% endlist %} 
 
{% list tabs group=instructions %} 
 
- Java 
 
  Текст таба 3. 
 
  * Можно использовать списки. 
  * И другую разметку. 
 
- Go 
 
  Текст таба 4. 
 
- C-sharp 
 
  Текст таба 4. 
 
{% endlist %}

@martyanovandrey
Copy link
Contributor

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

@vsesh
Copy link
Collaborator

vsesh commented Feb 26, 2025

Эта логика должна быть за пределами текущего модуля. Кто-то может захотеть сохранять историю в свою базу, или в куки, или из-за GTPR вообще ничего не сохранять на устройстве пользователя. Здесь нужно только данные для этого в событии предоставлять

bazuka5801 added a commit to diplodoc-platform/client that referenced this pull request Feb 26, 2025
@bazuka5801
Copy link
Contributor Author

@vsesh Waiting for your review 🙏

@bazuka5801 bazuka5801 merged commit 749a20b into master Mar 4, 2025
2 checks passed
@bazuka5801 bazuka5801 deleted the mbazuev/02-20/tabs-save-restore branch March 4, 2025 11:22
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

Successfully merging this pull request may close these issues.

3 participants