Ticket #1274 (closed defect: fixed)

Opened 4 years ago

Last modified 12 months ago

Can not erase bookmark

Reported by: erikos Owned by: humitos
Priority: Unspecified by Maintainer Milestone: 0.86
Component: Read Version: Git as of bugdate
Severity: Major Keywords: 12.1.0, patch
Cc: godiard, humitos Distribution/OS: Unspecified
Bug Status: Unconfirmed

Description

When I click on the bookmark to erase it, the dialog to edit the bookmark is displayed again.

Attachments

Change History

  Changed 4 years ago by sayamindu

Hmm - you need to click on the bookmark icon om the toolbar (again) to delete it (note that it remains active when the bookmark is around). Clicking on the bookmark just allows you to edit the notes that you associated with the bookmark.

Does it make sense to add a delete option to the edit-dialog that pops up ?

  Changed 13 months ago by greenfeld

I've tried clicking on the bookmark itself and that doesn't seem to delete it.

Tested in OLPC 12.1.0 os8/Read-99.

  Changed 13 months ago by greenfeld

  • cc godiard added

  Changed 13 months ago by godiard

The procedure (obviously not obvious) is click again in the star button in the toolbar.

Is harder to see now, because the toogle button is showed without the needed grey rounded square, but a almost black square background. This issue was already reported.

  Changed 13 months ago by godiard

  • keywords 12.1.0 added

follow-up: ↓ 8   Changed 13 months ago by godiard

  • owner changed from sayamindu to godiard
  • status changed from new to assigned

  Changed 13 months ago by garycmartin

Adding a standard Sugar Alert strip when deleting a bookmark would also provide some user feedback and a warning about the loss of data when your book mark is removed (e.g. your bookmark notes are lost). Also a future enhancement could be to show an optional tray across the bottom of the Read canvas listing all the pages you have bookmarked, like we do in Browse (and similar to Record).

in reply to: ↑ 6   Changed 13 months ago by godiard

Replying to godiard:

Also a future enhancement could be to show an optional tray across the bottom of the
Read canvas listing all the pages you have bookmarked, like we do in Browse (and
similar to Record).

Yes, will be better for consistency too.

  Changed 12 months ago by humitos

  • cc humitos added

  Changed 12 months ago by humitos

  • keywords 12.1.0, patch added; 12.1.0 removed
  • owner changed from godiard to humitos
  • status changed from assigned to accepted

I've already uploaded a patch. I would like to know if the text strings are right. I used "Delete bookmark" as title and "All the information related with this bookmark will be lost" as message.

  Changed 12 months ago by godiard

  • status changed from accepted to closed
  • resolution set to fixed

Thanks, pushed. Will be available in Read 100

Note: See TracTickets for help on using tickets.