fix updating mid-list

Author: shakyShane

special-pages/pages/new-tab/app/activity/ActivityProvider.js

@@ -295,7 +295,8 @@ export function SignalStateProvider({ children }) {
             urls.value = normalizeUrls(urls.value, evt.data);
             const visible = keys.value;
             const all = urls.value;
-            const nextVisibleRange = all.slice(0, Math.max(service.INITIAL, Math.min(service.INITIAL, visible.length)));
+            const nextVisibleRange = all.slice(0, Math.max(service.INITIAL, Math.max(service.INITIAL, visible.length)));
+            console.log({ nextVisibleRange });
             setVisibleRange(nextVisibleRange);
             fillHoles();
         });

special-pages/pages/new-tab/app/activity/batched-activity.service.js

@@ -220,18 +220,19 @@ export class BatchedActivityService {
      * @param {string} url
      */
     remove(url) {
-        this.dataService.update((old) => {
+        this.urlService.update((old) => {
+            const next = old.urls.filter((x) => x !== url);
             return {
                 ...old,
-                activity: old.activity.filter((item) => {
-                    return item.url !== url;
-                }),
+                urls: next,
             };
         });
-        this.urlService.update((old) => {
+        this.dataService.update((old) => {
             return {
                 ...old,
-                urls: old.urls.filter((x) => x !== url),
+                activity: old.activity.filter((item) => {
+                    return item.url !== url;
+                }),
             };
         });
         this.ntp.messaging.notify('activity_removeItem', { url });