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

Taphold action on oh-label-card and oh-button #1313

Merged
merged 1 commit into from
Mar 9, 2022
Merged

Conversation

tarag
Copy link
Contributor

@tarag tarag commented Feb 16, 2022

This PR provides another configurable action upon taphold (or calling contextual menu on desktop) in oh-label-cards and oh-buttons.

This is quite interesting for wall mounted interfaces when using canvas layout and a floor plan. A button controlling a light might do the basic action upon touch (toggle the light), and display a popup with additional actions upon hold (like dimming, color...).

…mponents upon tap and hold (or contextual menu call on desktop).

Signed-off-by: Gautier Taravella <tarag@mailbox.org>
@tarag tarag requested a review from a team as a code owner February 16, 2022 17:36
@tarag
Copy link
Contributor Author

tarag commented Feb 16, 2022

Somehow all the hard work seemed to be already done 👍 .

@relativeci
Copy link

relativeci bot commented Feb 16, 2022

Job #351: Bundle Size — 10.69MB (~+0.01%).

d042875 vs 3227215

Changed metrics (2/8)
Metric Current Baseline
Initial JS 1.67MB(+0.04%) 1.67MB
Cache Invalidation 18.52% 2.91%
Changed assets by type (1/7)
            Current     Baseline
JS 8.63MB (~+0.01%) 8.63MB

View Job #351 report on app.relative-ci.com

@ghys
Copy link
Member

ghys commented Mar 8, 2022

Thanks! For completeness sake, could you elaborate how to chose to add this to oh-label-card & oh-button only and not components which perhaps could benefit from this as well?

@tarag
Copy link
Contributor Author

tarag commented Mar 9, 2022

It's just a start to see how things goes (from a usage standpoint). I think these are the most generic components on which actions are used. In addition, there are others like toggle or roller shutter on which I don't think it would be a great choice to have this kind of action.

Copy link
Member

@ghys ghys left a comment

Choose a reason for hiding this comment

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

Alright, thanks!

@ghys ghys merged commit 9a19a25 into openhab:main Mar 9, 2022
ghys added a commit to ghys/openhab-webui that referenced this pull request Mar 21, 2022
Bug introduced in openhab#1313 and breaking config sheets in various parts of the UI.

Fixes openhab#1343.
Fixes  openhab/openhab-core#2850.

Signed-off-by: Yannick Schaus <github@schaus.net>
ghys added a commit that referenced this pull request Mar 21, 2022
Bug introduced in #1313 and breaking config sheets in various parts of the UI.

Fixes #1343.
Fixes  openhab/openhab-core#2850.

Signed-off-by: Yannick Schaus <github@schaus.net>
@ghys ghys added this to the 3.3 milestone May 1, 2022
@ghys ghys added enhancement New feature or request main ui Main UI labels May 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request main ui Main UI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants