-
-
Notifications
You must be signed in to change notification settings - Fork 24
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
Home Assistant 2022.11 compatibility #47
base: master
Are you sure you want to change the base?
Conversation
I think we can also edit the readme to add the new |
@piitaya Added, thanks for pointing it out! |
@piitaya or @ov1d1u, do you know if its possible to get the resulting stack-in-card.js file (or contents in a pastebin?) without waiting for the new release? I tried making the edits in my local file based on your source edits, but the release pipeline obfuscates the code and I'm not that experienced in javascript to be able to edit it myself 😅 |
@Getslow6 I added a compiled .js file to my repository, in the releases section, you can download it from https://github.com/ov1d1u/stack-in-card/releases/download/0.2.1/stack-in-card.js |
Can this please be merged? Thansk. |
Uh oh the documentation README has not been updated! |
Thanks for this! I'm just wondering what the exact syntax would be to prevent borders on child cards. I assume:
but that doesn't seem to work? |
Try removing the cache of your browser (not just reload). You don't need the
|
Nope, tried that already, but no difference. Thanks anyway. |
I overwrote the existing stack-in-card.js file with this one, cleared my browser cache, and tried many combinations of |
@strikeir13 Are you using HACS? Do you have a If so, you need to delete that, as HACS will serve up the gzipped version. After deleting the file, clear your cache again. |
That did it! Thank you! |
What's preventing this from getting merged? Can I be added to the reviewers list? Thanks. |
Any idea when this will be merged? |
Can you advise a novice user the steps? |
@Getslow6 I uploaded your compiled .js file from your repository and it works to remove the borders but can I use the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needs README changes please. Other than that you should have a green light from me.
@Rudd-O I also updated the README.md file a while ago to reflect the new |
@ov1d1u the |
@almighty059 I think you misunderstood the documentation a bit. The example given here regarding how to use
Anyway, I added support for the
I also updated the documentation to reflect the usage of the |
@ov1d1u thanks for the reply. Just to clarify, I was referring to the code in your |
This fix does not solve problems with a case "a stack inside another stack"- see #51 |
What am I doing wrong, I installed "custom:stack-in-card" it through HACS and gives it out when filling out the card |
It looks like this PR will never be merged, so I decided to remove the borders myself with some CSS, and it's actually very easy, you just need to install the Mod Card, and add the following to every stacked card: type: custom:stack-in-card
+ card_mod:
+ style: 'ha-card {--ha-card-border-width: 0}'
|
Why will this PR not get merged? It should! |
I'm just guessing based on the fact that this repo's last commit was in 2020 |
Does anyone know what happened to the maintainer of this project? |
Any news on probability of this getting merged? |
@ov1d1u
The upper button definitely has own border: But I cannot use
as well as with
Does it mean that your added The earlier added |
Home Assistant 2022.11 added borders around cards. This PR comes with the option to hide them.