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

add ns 7.0 compatibility #7

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Upd4ting
Copy link

feel free to review and accept it

@micc83
Copy link

micc83 commented Oct 5, 2020

@Upd4ting Thank you very much, your PR works like a charm (at least on Android, didn't try on iOs). Hopefully it will be merged soon.

@eaglejs
Copy link

eaglejs commented Dec 4, 2020

Any movement on this? Or is this plugin abandoned?

@Upd4ting
Copy link
Author

Upd4ting commented Dec 6, 2020

@eaglejs seems abandoned but you can use this npm package if you need the ns-7 version https://www.npmjs.com/package/nativescript-vue-shadow-ns-7

@eaglejs
Copy link

eaglejs commented Dec 6, 2020

Thank you @Upd4ting! Since my app is Android only, I ended up just using the androidElevation attribute/css property.
You can put it on any NS defined element. I decided to put it on a class .card in my css file.

.card { background-color: #fff; android-elevation: 5; }
alternatively, you can put it on the element as an attribute
<StackLayout androidElevation="5"></StackLayout>

That gave me the shadows back, only downside is that it does not work on iOS. And you do need a background-color defined.

@jawa-the-hutt
Copy link
Owner

Hey all. Sorry for not responding up to this point. Been trying to figure out a way to do this and make sure everything works. However, I created this plugin over a year ago, and have since lost access to a Mac to be able to do any further development and validation that things are working.

I'm not actually using Nativescript on anything at the moment due to a job change so that's another issue I have with trying to maintain this. With those two things together, I just haven't done anything with this PR and I apologize for that. It's not fair to all of you who use it.

I can go ahead and merge this PR without testing it, but not sure when I will be able to get npm updated. You can still install directly from Github but I know that's a pain.

So, the choices are:

  1. Let this repo and plugin die and everyone use the new fork.
  2. Merge the PR and hope it works for everyone and if it doesn't, then there may not be any further updates.
  3. Allow someone else in the community the ability to manage the repo and npm package.
  4. Turn the repo and npm over to someone who wants to maintain it going forward.

Looking for feedback on this.

@Upd4ting
Copy link
Author

Hello @jawa-the-hutt

No worries, thanks for your answer!
If you need help, I can maintain this for you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants