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

[Bug]: custom shortcuts and title overlap #212

Closed
amitkeret opened this issue Dec 16, 2021 · 27 comments
Closed

[Bug]: custom shortcuts and title overlap #212

amitkeret opened this issue Dec 16, 2021 · 27 comments
Labels
bug Something isn't working

Comments

@amitkeret
Copy link
Contributor

The new custom shortcuts added in PR #169 create a layout issue.
They overlap with the title, if it is configured.

image

type: custom:webrtc-camera
entity: camera.sd
title: Dining Room
ui: true
shortcuts:
  - name: Record
    icon: mdi:record-circle-outline
    service: switch.toggle
    service_data:
      entity_id: switch.camera_record
@AlexxIT AlexxIT added the bug Something isn't working label Dec 16, 2021
@AlexxIT
Copy link
Owner

AlexxIT commented Dec 16, 2021

@dmamontov

@dmamontov
Copy link
Contributor

Yeah, I'll fix it in the evening.

@dmamontov
Copy link
Contributor

Fixed) #214

@Napalmcold
Copy link

I did edited webrtc-camera.js accordingly to #214. Still can't change position of shortcut.
This config doesn't show any shotcuts:

type: custom:webrtc-camera
entity: camera.hall_camera
ui: true
shortcuts:
  top: 0
  left: 0
  orientation: vertical
  services:
    - name: Open
      icon: mdi:lock-open
      service: button.press
      service_data:
        entity_id: button.hall_unlocker
background: true

This one shows shortcut overlaped with title:

type: custom:webrtc-camera
entity: camera.ulitsa_camera
title: Вход с улицы
background: true
ui: true
shortcuts:
  - name: Open
    icon: mdi:lock-open
    service: button.press
    service_data:
      entity_id: button.ulitsa_unlocker

@amitkeret
Copy link
Contributor Author

@Napalmcold
seems like #214 hasn't been added to a release yet.
once @AlexxIT releases v2.2.1 you'll be able to pull it and use the new configuration
otherwise use master branch directly

@Napalmcold
Copy link

The deal is that I am using master. Removed webrtc integration completely And installed the master.

@dmamontov
Copy link
Contributor

dmamontov commented Jan 25, 2022

@Napalmcold Try clearing your browser cache. and top:0, left:0 have a minimum spacing of 13px.

@dmamontov
Copy link
Contributor

for example:
image

@Napalmcold
Copy link

for example: image

As soon as add lines "top, left, orientation, services", buttons just dissapear

@dmamontov
Copy link
Contributor

@Napalmcold The behavior is very similar to the fact that you have an old script either in hass or cached in the browser, check in another browser (in the browser, not in the application)

@Napalmcold
Copy link

Ok, I managed it. Indeed it was cache as for the Chrome browser and for the android app.
Now I've noticed another issue: position of shortcuts differs depending on resolution of the screen (laptop or mobile app).
Here are some example of the same card on different devices:
image
image

@dmamontov
Copy link
Contributor

@Napalmcold it is in pixels because. in theory, it can be modified so that it is possible to transfer the unit of measurement and some kind of% or vh.

@dmamontov
Copy link
Contributor

@Napalmcold in the next PR added the ability to use padding in other css units, be it % or vh #245

@Napalmcold
Copy link

@dmamontov, видимо можем по-русски)
Поправил код в соответствии с PR, но всё равно не могу понять, как указать положиние ровно по середине на разных экранах. Что px что vh делают одно и то же. Может нужно переставить master?

@dmamontov
Copy link
Contributor

dmamontov commented Jan 28, 2022

@Napalmcold

  1. Смотрим размер блока в режиме отладки в chrome
    image
  2. Добавляем расчет:
shortcuts:
  left: calc(50% - 12px - 48px)
  services:

где:
12px это отступ по умолчанию который всегда есть (оставляем как есть)
48px это половина ширины из 1 пункта.

@Napalmcold
Copy link

@dmamontov, а 50% от чего он считает? При таком коде у меня иконка просто упирается в левый край камеры

image

@dmamontov
Copy link
Contributor

dmamontov commented Jan 28, 2022

@Napalmcold calc пропустил

по умолчанию 50% от (100% - 12px)

@Napalmcold
Copy link

@Napalmcold calc пропустил

по умолчанию 50% от (100% - 12px)

Это я уже играться пытался, но calc ничего не меняет. И это не из-за кэша. проверил простые числовые значения - положение меняется.
image

@dmamontov
Copy link
Contributor

dmamontov commented Jan 28, 2022

@Napalmcold попробуй переставить с мастера либо удалить webrtc-camera.js.gz. скорее всего тянется он. по скрину явно просто ломается css

@Napalmcold
Copy link

@dmamontov то есть всё удаляю и заново ставлю master? без ручной корректировки файла?

@dmamontov
Copy link
Contributor

@Napalmcold для начала переставь, если не поможет то переименуй файл gz, если тоже нет то проверь в режиме инкогнито.

@Napalmcold
Copy link

@dmamontov всё, связанное с webrtc снёс, поставил master - вообще шорткаты пропали, через инкогнито тоже)))

@Napalmcold
Copy link

Переставил еще раз. Шорткаты появились, но всё так же жмутся к левому краю. Как будто с формулой что-то не так

@dmamontov
Copy link
Contributor

@Napalmcold специально переставил и все работает) Какая конфигурация сейчас?

@dmamontov
Copy link
Contributor

dmamontov commented Jan 28, 2022

Так, ну проверил c left: calc(50% - 12px - 48px):
image

важно чтобы был calc и пробелы внутри calc(специфика css)

@Napalmcold
Copy link

@dmamontov Не знаю, что поменялось, но щас выходил из дома, пришёл, увидел твоё сообщение, скопипастил и всё пошло))) Спасибо!

@AlexxIT
Copy link
Owner

AlexxIT commented Apr 11, 2022

All fixes in latest release https://github.com/AlexxIT/WebRTC/releases/tag/v2.3.0
Please reopen issue if it still actual

@AlexxIT AlexxIT closed this as completed Apr 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants