martin
Avril 29, 2020, 2:21
122
Une autre mise à jour : ces deux PR sont désormais fusionnées et il ne reste aucun bug en suspens. Je travaille actuellement sur une dernière PR pour renommer des variables, etc., comme bookmarkWithReminder en bookmark.
master ← issue/cancel-bookmark-save-if-modal-close-clicked
merged 10:45PM - 28 Apr 20 UTC
* After this change the bookmark will still be saved if clicking out of the moda… l or pressing escape
* To achieve this I implemented an initiatedBy parameter for modal closing from d-modal. If clicking on the cross it is initiated by close, if clicking out of the modal it is by click out.
* These options can then be compared in controllers consuming `onClose`
master ← issue/change-bookmarks-with-reminders-route-back-to-bookmarks
merged 12:53AM - 29 Apr 20 UTC
* Bookmarks with reminders is a core feature now, no need to have a separate URL…
* Keep around the old /u/:username/activity/bookmarks-with-reminders route for backwards compat in Ember but just redirect to user activity bookmarks.
12 « J'aime »