lightweight-charts icon indicating copy to clipboard operation
lightweight-charts copied to clipboard

New type of Markers: Icons

Open Flydexo opened this issue 3 years ago • 6 comments

Type of PR: enhancement

Before creating a complete pull request with tests and the documentation update. I just wanted to know if you agree with this feature.

Flydexo avatar Jan 12 '22 11:01 Flydexo

Before creating a complete pull request with tests and the documentation update. I just wanted to know if you agree with this feature.

Interesting approach to ask whether we agree with this feature by implementing it and providing a PR 😄

@Flydexo do have examples how it looks like with different images?

timocov avatar Jan 12 '22 16:01 timocov

Yep there you go market cap dark btc candle dark btc candle white market cap white

Flydexo avatar Jan 12 '22 18:01 Flydexo

Code example: {time: Math.round(date/1000), position: "aboveBar", shape: "icon", size: 5, id: a.id, image: logo}

Flydexo avatar Jan 12 '22 18:01 Flydexo

@timocov

Flydexo avatar Jan 13 '22 15:01 Flydexo

This would be nice to have, any updates on this?

fillky avatar Apr 27 '22 07:04 fillky

Hi @Flydexo!

Apologies for missing this one out :( Would you mind rebasing your PR and making sure it compiles OK? We would then review it and check if everything is fine before merging it.

romfrancois avatar Apr 28 '22 14:04 romfrancois

Any updates on this being merged in? @romfrancois @timocov @Flydexo 🙏

aliasmac avatar Nov 01 '22 11:11 aliasmac

As Romain's last comment mentions the merge conflicts would need to be solved first.

Some graphics tests should also be added I think.

Once those two things are done we can start a proper review.

edew avatar Nov 08 '22 10:11 edew

Thank you for your feature PR for Lightweight Charts. In order to keep the library lightweight and maintain a narrow focus, we strive to include only strictly required features in the core library. As a result, we have decided that this feature should be implemented as a plugin instead, and will be closing this PR as a result.

We recently introduced the ability to write plugins for Lightweight Charts, allowing users like you to extend the functionality of the library without adding it to the core. We highly recommend implementing your feature request as a plugin to achieve the desired functionality while keeping the core bundle size smaller for all users.

To get started with plugins, please refer to our Plugins Documentation for a better understanding of what is possible and how plugins work. You can also explore our collection of plugin examples (with a preview hosted here) for inspiration and guidance on implementing specific functionality.

To help you get started quickly, we have created an NPM package called create-lwc-plugin, which sets up a plugin project for you. This way, you can hit the ground running with your plugin development.

If you do create a plugin based on your feature request, we would appreciate it if you could publish it on GitHub or NPM and let us know. This way, we can share it with the rest of the community. It would be helpful if you could prefix your NPM package name with lwc-plugin- for easier discoverability by others.

Thank you for your understanding and support. We believe implementing your feature as a plugin will greatly benefit both you and the Lightweight Charts community.

SlicedSilver avatar Oct 04 '23 14:10 SlicedSilver