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.

Sign up for GitHub

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

Jump to bottom

Implements the basemap selector as anywidget #2171

Merged
merged 46 commits into from
Nov 18, 2024
Merged

Implements the basemap selector as anywidget #2171

merged 46 commits into from
Nov 18, 2024

Conversation

sufyanAbbasi
Copy link
Collaborator

@sufyanAbbasi sufyanAbbasi commented Nov 14, 2024 •
edited
Loading

  • Converts the basemap selector to anywidget.
  • Renames Basemap --> BasemapSelector, with alias.

image

naschmitz and others added 30 commits October 10, 2024 13:17
@naschmitz
Implement LayerManager using LitElement + anywidget
f2b7037
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
8e3ee04 
for more information, see https://pre-commit.ci
@giswqs
Merge branch 'master' into anywidget
acde734
@naschmitz
Update static files
919e39a
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
f5af00d 
for more information, see https://pre-commit.ci
@naschmitz
Use non-minified JS files to work around property renaming issue
b87f39d
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
6381d46 
for more information, see https://pre-commit.ci
@sufyanAbbasi
Set up tests for layer_manager_row
c31d5a9
@sufyanAbbasi
Set up layer_manager_row test
51a8556
@naschmitz
Implement LayerManager using LitElement + anywidget
cdd5040
@pre-commit-ci @naschmitz
[pre-commit.ci] auto fixes from pre-commit.com hooks
bbfa97d 
for more information, see https://pre-commit.ci
@naschmitz
Update static files
5fa9733
@pre-commit-ci @naschmitz
[pre-commit.ci] auto fixes from pre-commit.com hooks
98ef790 
for more information, see https://pre-commit.ci
@naschmitz
Use non-minified JS files to work around property renaming issue
0729c3d
@pre-commit-ci @naschmitz
[pre-commit.ci] auto fixes from pre-commit.com hooks
b9e0d69 
for more information, see https://pre-commit.ci
@naschmitz
Clean up setuptools references in pyproject.toml
7195ac8
@naschmitz
Clean up setuptools references in pyproject.toml
ce4c34d
@naschmitz
Fix dark mode and drop shadow issues in Colab
28092e7
@sufyanAbbasi
Remove common.css, load fonts using JS instead.
78e0021
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
9d359bc 
for more information, see https://pre-commit.ci
@sufyanAbbasi
Merge branch 'anywidget' into anywidget-test
8325c5a
@sufyanAbbasi
Rebuild
301d254
@sufyanAbbasi
Remove extraneous files
0d20641
@naschmitz
Address comments from initial review
84f2661
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
4a462c0 
for more information, see https://pre-commit.ci
@naschmitz
Ignore static files
befdaa7
@sufyanAbbasi
Fix TS errors
fc9952b
@sufyanAbbasi
Merge branch 'anywidget' into anywidget-test
7c1d6fc
@giswqs
Merge branch 'master' into anywidget
243c178
@naschmitz
Convert tsconfig.json to spaces and export model interfaces
25817a7
naschmitz
naschmitz requested changes Nov 15, 2024
View reviewed changes
Copy link
Collaborator

@naschmitz naschmitz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great! Just some minor nits. Thank you!

Let's wait for Qiusheng to look at this before submitting.

geemap/map_widgets.py Show resolved Hide resolved
js/basemap_selector.ts Outdated Show resolved Hide resolved
js/basemap_selector.ts Outdated Show resolved Hide resolved
js/basemap_selector.ts Outdated Show resolved Hide resolved
js/basemap_selector.ts Show resolved Hide resolved
js/basemap_selector.ts Show resolved Hide resolved
js/ipywidgets_styles.ts Show resolved Hide resolved
geemap/core.py Outdated Show resolved Hide resolved
js/layer_manager copy.ts Outdated Show resolved Hide resolved
js/basemap_selector.ts Outdated Show resolved Hide resolved
@sufyanAbbasi sufyanAbbasi mentioned this pull request Nov 15, 2024
Merged
giswqs
giswqs approved these changes Nov 18, 2024
View reviewed changes
Copy link
Member

@giswqs giswqs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Excellent work! It works very well

image

@giswqs giswqs merged commit b0bf541 into gee-community:master Nov 18, 2024
15 checks passed
@naschmitz naschmitz deleted the anywidget-basemap-selector branch November 18, 2024 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@naschmitz naschmitz naschmitz requested changes

@giswqs giswqs giswqs approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sufyanAbbasi @giswqs @naschmitz