guim31
11. September 2023 um 18:07
1
Ich habe gerade ein Problem (auf mobilen und Desktop-Geräten) bei der Erstellung einer Szene festgestellt.
Früher, als ich etwas HTML kannte, hätte ich an ein Problem mit dem z-index gedacht (aber ich weiß nicht mal, ob das noch existiert ^^), denn die Dropdowns überlappen sich.
Hier ist etwas, um meine Aussage zu veranschaulichen: https://streamable.com/e/9hmgrl (dieser Link läuft in 48 Stunden ab … Hinweis an @Pierre-Gilles , falls du in den nächsten zwei Tagen vorbeikommst !!)
Danke für das Feedback @guim31 ! Könntest du ein GitHub-Issue mit dem Video erstellen?
Übrigens, ich möchte klarstellen, dass das Forum kleine .mp4-Dateien akzeptiert, also wäre dieses Video problemlos im Forum gelaufen
guim31
12. September 2023 um 07:50
3
Ich werde heute ein Issue erstellen, kein Problem.
Das Problem ist, dass das Video im .webm-Format ist und ich daran nicht gedacht habe, es zu konvertieren ^^
@guim31 meinst du das?
Denn ich habe die Anzeige des Dropdown-Menüs… von der unteren Box, die vor der Liste angezeigt wird.
Entschuldigung für das Foto, ein Screenshot war nicht möglich.
guim31
2. November 2023 um 07:49
8
Will_71
21. November 2023 um 22:18
9
@pierre-gilles
Hier ist der PR
master ← William-De71:scene-trigger-selectDeviceFeature
ouvert 10:12PM - 21 Nov 23 UTC
### Pull Request check-list
To ensure your Pull Request can be accepted as fa… st as possible, make sure to review and check all of these items:
- [ ] If your changes affects code, did your write the tests?
- [ ] Are tests passing? (`npm test` on both front/server)
- [ ] Is the linter passing? (`npm run eslint` on both front/server)
- [ ] Did you run prettier? (`npm run prettier` on both front/server)
- [ ] If you are adding a new features/services, did you run integration comparator? (`npm run compare-translations` on front)
- [ ] Did you test this pull request in real life? With real devices? If this development is a big feature or a new service, we recommend that you provide a Docker image to the community ([french forum](https://community.gladysassistant.com/)/[english forum](https://en-community.gladysassistant.com/)) for testing before merging.
- [ ] If your changes modify the API (REST or Node.js), did you modify the API documentation? (Documentation is based on comments in code)
- [ ] If you are adding a new features/services which needs explanation, did you modify the user documentation? See [the GitHub repo](https://github.com/GladysAssistant/v4-website) and the [website](https://gladysassistant.com).
- [ ] Did you add fake requests data for the demo mode (`front/src/config/demo.js`) so that the demo website is working without a backend? (if needed) See [https://demo.gladysassistant.com](https://demo.gladysassistant.com).
NOTE: these things are not required to open a PR and can be done afterwards / while the PR is open.
### Description of change
Fix z-index issue in selectDeviceFeature on device change in scene trigger
https://github.com/GladysAssistant/Gladys/issues/1923

Will_71
21. November 2023 um 22:45
10
Ähnlicher Bug mit der Aktion „Nur fortsetzen, wenn“ in Szenen
Will_71
22. November 2023 um 08:59
11
Das GitHub-Issue
ouvert 10:42PM - 21 Nov 23 UTC
fermé 10:00AM - 27 Nov 23 UTC
z-issue in scene action only continue if

EDIT : @pierre-gilles
Der PR
master ← William-De71:scene-action-condition
ouvert 09:12PM - 22 Nov 23 UTC
### Pull Request check-list
To ensure your Pull Request can be accepted as fa… st as possible, make sure to review and check all of these items:
- [ ] If your changes affects code, did your write the tests?
- [ ] Are tests passing? (`npm test` on both front/server)
- [ ] Is the linter passing? (`npm run eslint` on both front/server)
- [ ] Did you run prettier? (`npm run prettier` on both front/server)
- [ ] If you are adding a new features/services, did you run integration comparator? (`npm run compare-translations` on front)
- [ ] Did you test this pull request in real life? With real devices? If this development is a big feature or a new service, we recommend that you provide a Docker image to the community ([french forum](https://community.gladysassistant.com/)/[english forum](https://en-community.gladysassistant.com/)) for testing before merging.
- [ ] If your changes modify the API (REST or Node.js), did you modify the API documentation? (Documentation is based on comments in code)
- [ ] If you are adding a new features/services which needs explanation, did you modify the user documentation? See [the GitHub repo](https://github.com/GladysAssistant/v4-website) and the [website](https://gladysassistant.com).
- [ ] Did you add fake requests data for the demo mode (`front/src/config/demo.js`) so that the demo website is working without a backend? (if needed) See [https://demo.gladysassistant.com](https://demo.gladysassistant.com).
NOTE: these things are not required to open a PR and can be done afterwards / while the PR is open.
### Description of change
Fix z-index issue in only-continue-if selector in scene action

Will_71
27. November 2023 um 12:27
13
Ich schaue mir das heute Abend an.
Ich habe deine Rezension gesehen, wenn du alles getestet hast und nichts kaputt geht, vertraue ich dir
Ich habe auf master gemerged!