)]}' { "commit": "3a7452e555c773e63e2f388b432c8a1346b7a7ea", "tree": "a5beb9ee5cbf545e8039f9973045cf6f21e8ff8e", "parents": [ "3c3703b0b7042ebb4f48a73bf48648358ef7cf07" ], "author": { "name": "Ben Rohlfs", "email": "brohlfs@google.com", "time": "Wed Jan 18 12:56:34 2023 +0000" }, "committer": { "name": "Ben Rohlfs", "email": "brohlfs@google.com", "time": "Wed Jan 18 14:34:45 2023 +0100" }, "message": "Revert \"Use router to set url instead of href\"\n\nThis reverts commit 8a8f02c927fd221a4dc02e6d3adf9b0e06b64309.\n\nReason for revert: We prefer normal links and are fixing the issue\nwith the full page reload in a different way: For button clicks we\nwill only handle direct clicks on its element, not its children.\n`stopPropagation` for `\u003ca\u003e` was causing issue, because then the\ntop-level page.js click handler would not pick up the click. It\nnormally intercepts `\u003ca\u003e` clicks and thus prevents full page\nreloads\n\nRelease-Notes: skip\nChange-Id: I4061868a0bd3a3d270a5a095c97a9bff7c840b9f\n", "tree_diff": [ { "type": "modify", "old_id": "c0ed3b36351098ca7bf6bd25b613652fce827b80", "old_mode": 33188, "old_path": "polygerrit-ui/app/elements/change/gr-change-view/gr-change-view.ts", "new_id": "6c1b681a4f00f50c2d6598a3375b49424104816a", "new_mode": 33188, "new_path": "polygerrit-ui/app/elements/change/gr-change-view/gr-change-view.ts" } ] }