developmentseed / developmentseed/lonboard

Remove NextUI dependency (and maybe Tailwind)

Open
#763 0 comments 2 reactions 1 assignee Claimed by @vgeorge View on GitHub
Dominant language
Python
Stars
964
Forks
54
Avg merge
5m
Merged PRs (30d)
1

Description

In https://github.com/developmentseed/lonboard/pull/740 we worked on a fix to avoid changing the environment page style caused by NextUI.

In that PR we moved the `tailwind @base` directive, which injects NextUI plugin styles, to the scope of the `.lonboard` div. We also added some hard-coded styles from NextUI to that div.

Considering that a change in the NextUI package might break this implementation, this is not a very maintainable solution. Also, we don't expect many different components to be present in the widget, so a component library might be overkill.

At the moment, I'm leaning towards removing it as a dependency and implementing custom styles, relying only on Tailwind or even removing it to avoid unintended side effects.

cc @kylebarron @Bjoern-Rapp

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.