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

Plugin refactoring prerel #34

Merged
merged 16 commits into from
Apr 25, 2024
Merged

Conversation

alxart
Copy link
Contributor

@alxart alxart commented Apr 16, 2024

Важное

Изменения:

  • Т.к. опцию preserveCache в методе removeRealtyScene удалил ранее, то теперь показалось все-таки необходимым наличие методов hideRealtyScene и showRealtyScene. Добавил эти методы.
  • Поправил баг с неисчезновением фасада здания при ините сцены недвижки с кастомным билдинг стейтом
  • Поправил баг с преждевременным появлением фасадов зданий при переключении с подземного этажа на любую неготовую модель
  • Поддержал высоту и интерактивность подписей
  • Добавил labelGroupDefaults в опции плагина - это аналог poiConfig в предыдущей версии. Туда же добавил дефолт для подложки подписей группы, который обсуждали тут
  • Поправил релизный пайплайн: версию 1 перевел на отдельный тег
  • Докинул опцию interactive в подпись и группу

@alxart
Copy link
Contributor Author

alxart commented Apr 21, 2024

@Farfurix глянь еще раз, плз: поправил пару багов и добавил методы show и hide для сцены недвижки

@alxart alxart requested a review from Farfurix April 21, 2024 18:45
Copy link
Contributor

@Farfurix Farfurix left a comment

Choose a reason for hiding this comment

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

Может добавим в демку hide и show перед Remove Scene?

    new mapglAPI.Control(map, '<button>Hide Scene</button>', {
        position: 'topLeft',
    })
        .getContainer()
        .addEventListener('click', () => {
            plugin.hideRealtyScene();
        });

    new mapglAPI.Control(map, '<button>Show Scene</button>', {
        position: 'topLeft',
    })
        .getContainer()
        .addEventListener('click', () => {
            plugin.showRealtyScene();
        });

image

test/screenshots/plugin.screen.ts Show resolved Hide resolved
@alxart
Copy link
Contributor Author

alxart commented Apr 22, 2024

Может добавим в демку hide и show перед Remove Scene?

    new mapglAPI.Control(map, '<button>Hide Scene</button>', {
        position: 'topLeft',
    })
        .getContainer()
        .addEventListener('click', () => {
            plugin.hideRealtyScene();
        });

    new mapglAPI.Control(map, '<button>Show Scene</button>', {
        position: 'topLeft',
    })
        .getContainer()
        .addEventListener('click', () => {
            plugin.showRealtyScene();
        });

image

✔️

@kalyanov
Copy link
Contributor

kalyanov commented Apr 23, 2024

В демке в первой сцене косяк с высотой поевин для вот этого этажа

Screenshot 2024-04-23 at 21 30 41

@kalyanov
Copy link
Contributor

Запротектил ветки master и plugin-v1

@alxart
Copy link
Contributor Author

alxart commented Apr 24, 2024

В демке в первой сцене косяк с высотой поевин для вот этого этажа

А это 100% воспроизводится? Не смог отловить у себя ни в одном из браузеров и сценариях переключения сцены.
Группа подписей точно нужная, т.е. она не с нижнего этажа берется. А у второго здания 11 этаж норм?

@alxart alxart merged commit 93dea4f into plugin-refactoring Apr 25, 2024
3 checks passed
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