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

Modelview deletekey functionality #831

Merged
merged 5 commits into from
Feb 4, 2021
Merged

Conversation

bigbasec
Copy link
Contributor

As I was testing my OH setup I found myself always winding up with orphan items in the model view, and clicking to remove them was a bit of a hassle, so I came up with this.

Might be a better way to do this, just really copied the code from the detail view and refactored for the main page.

@bigbasec bigbasec requested a review from a team as a code owner January 21, 2021 14:44
@bigbasec bigbasec force-pushed the modelview_deletekey branch from c654c16 to f5b0a63 Compare January 21, 2021 16:53
@ghys
Copy link
Member

ghys commented Jan 24, 2021

Since the delete logic is already in item-details.vue maybe it would be better to add the handler there instead of in model.vue?

@bigbasec
Copy link
Contributor Author

100% first time ever doing anything w/ F7 or Vue, so little bit of a learning curve here, appologies. I'll take a look at that though and see what I can come up with. And yes, this was literally a copy/paste from there and changing it to match the vue.

Signed-off-by: Brian Homeyer <bhomeyer@gmail.com>
Signed-off-by: Brian Homeyer <bhomeyer@gmail.com>
This reverts commit f5b0a63.

Signed-off-by: Brian Homeyer <bhomeyer@gmail.com>
This reverts commit a5e4328.

Signed-off-by: Brian Homeyer <bhomeyer@gmail.com>
Move changes to item-detail view

Signed-off-by: Brian Homeyer <bhomeyer@gmail.com>
@bigbasec bigbasec force-pushed the modelview_deletekey branch from 8b7f81f to 555dd16 Compare January 28, 2021 20:29
@bigbasec
Copy link
Contributor Author

@ghys Should be all set now. Thanks

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.

LGTM now!

@ghys ghys merged commit 61eb852 into openhab:main Feb 4, 2021
@ghys ghys added enhancement New feature or request main ui Main UI labels Feb 4, 2021
@ghys ghys added this to the 3.1 milestone Feb 4, 2021
@bigbasec bigbasec deleted the modelview_deletekey branch February 5, 2021 14:33
@ghys ghys added the patch A PR that has been cherry-picked to a patch release branch label Apr 18, 2021
ghys pushed a commit that referenced this pull request Apr 18, 2021
Signed-off-by: Brian Homeyer <bhomeyer@gmail.com>
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 patch A PR that has been cherry-picked to a patch release branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants