martin
(Martin Brennan)
29.Апрель.2020 02:21:43
122
Ещё одно обновление — оба этих PR уже объединены, и неполадки устранены. Сейчас я работаю над финальным PR, чтобы переименовать переменные и прочее, например, bookmarkWithReminder обратно в 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 лайков