The issue has been solved

chore: Replace vue-lazyload with a custom library #196

krmax44 posted onGitHub

Feature request

Replace vue-lazyload with a custom library. Related #197

What problem does this feature solve?

vue-lazyload isn't properly maintained and is missing some features like events plus it has a couple of weird bugs.

What does the proposed API look like?

Simple API with events and stricter configuration - vue-lazyload's one is pretty loose with too many options that can cause conflicts or straight up don't work sometimes.

How should this be implemented in your opinion?

Either straight in SaberImage or as a seperate library. Thoughts?

Are you willing to work on this yourself?

Yes.

I guess it can be a separate library so that other Vue projects can use it as well

posted by egoist almost 6 years ago

@issuehunt has funded $100.00 to this issue.

Submit pull request via IssueHunt to receive this reward.
Want to contribute? Chip in to this issue via IssueHunt.
Checkout the IssueHunt Issue Explorer to see more funded issues.
Need help from developers? Add your repository on IssueHunt to raise funds.

posted by IssueHuntBot almost 6 years ago

@egoist Should it still go into the monorepo or be a seperate repo?

posted by krmax44 almost 6 years ago

A separate one.

Max notifications@github.com 于 2019年5月27日周一上午2:58写道：

@egoist https://github.com/egoist Should it still go into the monorepo or be a seperate repo?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/saberland/saber/issues/196?email_source=notifications&email_token=ACDAWSDXA6XKPPF5CYGQJE3PXLMWDA5CNFSM4HN5AGY2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWILM2I#issuecomment-496023145, or mute the thread https://github.com/notifications/unsubscribe-auth/ACDAWSEYUJ3Q4A7LGHDZEGTPXLMWDANCNFSM4HN5AGYQ .

posted by egoist almost 6 years ago

Superseeded by #356

posted by krmax44 over 5 years ago