Skip to content

Commit 4456237

Browse files
committed
Fix #3416 Add snackbar on clicking add/remove from bookmarks
1 parent fd706cf commit 4456237

File tree

3 files changed

+15
-1
lines changed

3 files changed

+15
-1
lines changed

app/src/main/java/fr/free/nrw/commons/bookmarks/pictures/BookmarkPicturesDao.java

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,18 @@
11
package fr.free.nrw.commons.bookmarks.pictures;
22

3+
import android.app.Activity;
34
import android.content.ContentProviderClient;
45
import android.content.ContentValues;
6+
import android.content.Intent;
57
import android.database.Cursor;
68
import android.database.sqlite.SQLiteDatabase;
79
import android.os.RemoteException;
10+
import android.view.View;
811

912
import androidx.annotation.NonNull;
1013

14+
import com.google.android.material.snackbar.Snackbar;
15+
1116
import java.util.ArrayList;
1217
import java.util.List;
1318

@@ -16,7 +21,9 @@
1621
import javax.inject.Provider;
1722
import javax.inject.Singleton;
1823

24+
import fr.free.nrw.commons.R;
1925
import fr.free.nrw.commons.bookmarks.Bookmark;
26+
import fr.free.nrw.commons.bookmarks.BookmarksActivity;
2027

2128
import static fr.free.nrw.commons.bookmarks.pictures.BookmarkPicturesContentProvider.BASE_URI;
2229

app/src/main/java/fr/free/nrw/commons/media/MediaDetailPagerFragment.java

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,8 @@
2020
import androidx.fragment.app.FragmentStatePagerAdapter;
2121
import androidx.viewpager.widget.ViewPager;
2222

23+
import com.google.android.material.snackbar.Snackbar;
24+
2325
import javax.inject.Inject;
2426
import javax.inject.Named;
2527

@@ -148,7 +150,9 @@ public boolean onOptionsItemSelected(MenuItem item) {
148150
Media m = provider.getMediaAtPosition(pager.getCurrentItem());
149151
switch (item.getItemId()) {
150152
case R.id.menu_bookmark_current_image:
151-
bookmarkDao.updateBookmark(bookmark);
153+
boolean bookmarkExists = bookmarkDao.updateBookmark(bookmark);
154+
Snackbar snackbar = bookmarkExists ? Snackbar.make(getView(), R.string.add_bookmark, Snackbar.LENGTH_LONG) : Snackbar.make(getView(), R.string.remove_bookmark, Snackbar.LENGTH_LONG);
155+
snackbar.show();
152156
updateBookmarkState(item);
153157
return true;
154158
case R.id.menu_share_current_image:

app/src/main/res/values/strings.xml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -591,4 +591,7 @@ Upload your first media by tapping on the add button.</string>
591591

592592
<string name="upload_nearby_place_found_title">Nearby Place Found</string>
593593
<string name="upload_nearby_place_found_description">Is this a photo of Place %1$s?</string>
594+
<string name="remove_bookmark">Removed from bookmarks</string>
595+
<string name="add_bookmark">Added to bookmarks</string>
596+
<string name="open">OPEN</string>
594597
</resources>

0 commit comments

Comments
 (0)