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

[BasicUI] Mapview elements in sitemap are always shown with same height #1064

Closed
1 task
schmidmuc opened this issue May 20, 2021 · 1 comment · Fixed by #1065
Closed
1 task

[BasicUI] Mapview elements in sitemap are always shown with same height #1064

schmidmuc opened this issue May 20, 2021 · 1 comment · Fixed by #1065
Labels
basic ui Basic UI bug Something isn't working

Comments

@schmidmuc
Copy link

Which UI are you reporting an issue for?

  • Basic UI

The problem

Mapview elements in sitemap are always shown with same height and cannot be changed by sitemap configuration.

Sitemap element:
Mapview item=ITEMNAME height=5

Expected behavior

Height is changed according to configuration parameter "height"

Steps to reproduce

  1. Create a mapview element in your sitemap: Mapview item=ITEMNAME height=5
  2. Visit your sitemap and you will see that height is smaller and cannot be modified

Your environment

  version: 3.1.0.M4
  buildString: Milestone Build
locale: de-DE
systemInfo:
  configFolder: /etc/openhab
  userdataFolder: /var/lib/openhab
  logFolder: /var/log/openhab
  javaVersion: 11.0.11
  javaVendor: Azul Systems, Inc.
  javaVendorVersion: Zulu11.48+21-CA
  osName: Linux
  osVersion: 5.4.0-40-generic
  osArchitecture: amd64
  availableProcessors: 4
  freeMemory: 1034848256
  totalMemory: 6121586688
bindings:
  - airquality
  - astro
  - avmfritz
  - bmwconnecteddrive
  - chromecast
  - dwdpollenflug
  - dwdunwetter
  - exec
  - gpstracker
  - http
  - ipp
  - ism8
  - keba
  - lgwebos
  - miio
  - mqtt
  - network
  - ntp
  - openweathermap
  - pushover
  - samsungtv
  - shelly
  - snmp
  - spotify
  - systeminfo
  - tankerkoenig
  - tplinksmarthome
  - tr064
  - wifiled
  - zwave
clientInfo:
  device:
    ios: false
    android: false
    androidChrome: false
    desktop: true
    iphone: false
    ipod: false
    ipad: false
    edge: false
    ie: false
    firefox: false
    macos: false
    windows: true
    cordova: false
    phonegap: false
    electron: false
    nwjs: false
    webView: false
    webview: false
    standalone: false
    os: windows
    pixelRatio: 1.25
    prefersColorScheme: light
  isSecureContext: false
  locationbarVisible: true
  menubarVisible: true
  navigator:
    cookieEnabled: true
    deviceMemory: N/A
    hardwareConcurrency: 4
    language: de-DE
    languages:
      - de-DE
      - de
      - en-US
      - en
    onLine: true
    platform: Win32
  screen:
    width: 1536
    height: 864
    colorDepth: 24
  support:
    touch: false
    pointerEvents: true
    observer: true
    passiveListener: true
    gestures: false
    intersectionObserver: true
  themeOptions:
    dark: light
    filled: true
    pageTransitionAnimation: default
    bars: filled
    homeNavbar: default
    homeBackground: default
    expandableCardAnimation: default
  userAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML,
    like Gecko) Chrome/90.0.4430.212 Safari/537.36
timestamp: 2021-05-20T20:54:54.493Z

Browser console


Browser network traffic


Additional information


@schmidmuc schmidmuc added the bug Something isn't working label May 20, 2021
@schmidmuc schmidmuc changed the title Mapview elements in sitemap are always shown with same height [BasicUI] Mapview elements in sitemap are always shown with same height May 20, 2021
@ghys ghys added the basic ui Basic UI label May 21, 2021
@lolodomo
Copy link
Contributor

After testing and checking the code, I can confirm that the height parameter is ignored in Basic UI.
The enhancement may be very easy, I will try.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
basic ui Basic UI bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants