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

Allow to star maps and retrieve starred maps #683

Merged
merged 6 commits into from
May 15, 2023
Merged

Allow to star maps and retrieve starred maps #683

merged 6 commits into from
May 15, 2023

Conversation

yohanboniface
Copy link
Member

@yohanboniface yohanboniface commented Apr 9, 2019

Not sure we really need this actually, but here is a POC.

Screenshot-20190409094456-2524x670
Screenshot-20190409094419-644x498

@yohanboniface yohanboniface changed the title (WIP) Allow to star map and retrieve starred maps (WIP) Allow to star maps and retrieve starred maps Apr 9, 2019
@Binnette Binnette changed the title (WIP) Allow to star maps and retrieve starred maps WIP: Allow to star maps and retrieve starred maps Feb 29, 2020
Copy link
Contributor

Choose a reason for hiding this comment

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

Missing white star?

Copy link
Member Author

Choose a reason for hiding this comment

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

I'd say we don't need those, because there are no actions nor button in the "edit area" (which is white on black), but I can use those just in case if you think that's more consistent.

Copy link
Contributor

Choose a reason for hiding this comment

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

Missing stars?

umap/static/umap/js/umap.controls.js Outdated Show resolved Hide resolved
umap/views.py Outdated Show resolved Hide resolved
umap/views.py Outdated Show resolved Hide resolved
umap/views.py Outdated Show resolved Hide resolved
@yohanboniface yohanboniface changed the title WIP: Allow to star maps and retrieve starred maps Allow to star maps and retrieve starred maps May 12, 2023
@davidbgk
Copy link
Contributor

See also #1084

@yohanboniface yohanboniface merged commit 317a8ba into master May 15, 2023
@yohanboniface yohanboniface deleted the fav branch May 15, 2023 13:40
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.

None yet

2 participants