)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":27952,"name":"Felix Edel","email":"felix.edel@bmw.de","username":"felix.schmidt"},"change_message_id":"8317d2d8a0fd07aa7f9d279b5af508518a768644","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"4423670b_c3024683","updated":"2024-07-04 05:22:16.000000000","message":"\u003e I don\u0027t think we should merge this change without the toggle, and since it exists now, I think you should squash the toggle change into this one.\n\nI thought about this as well - and I think I even tried it - but I would have to reorder some of the changes in order to do so (some of the changes in between this change and the \"Add switch\" change are needed for the latter one, so I would have to integrate them as well or move them down up in the stack. Maybe I can try to include all the \"improve/align layout\" changes into their corresponding changes where the component was introduced.","commit_id":"4bf74f8272856ce91b9439a7007dd880ec027118"},{"author":{"_account_id":27952,"name":"Felix Edel","email":"felix.edel@bmw.de","username":"felix.schmidt"},"change_message_id":"fc95d564d861ffcf2677fea342bc17d0e49dbe11","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"a3f05705_400b5f26","updated":"2024-09-09 05:25:15.000000000","message":"recheck","commit_id":"48c44d9ea37c849937b5de64fe094c0fa8294b3a"}],"web/src/pages/PipelineOverview.jsx":[{"author":{"_account_id":27952,"name":"Felix Edel","email":"felix.edel@bmw.de","username":"felix.schmidt"},"change_message_id":"0d53bbbc666f82b7fd8d94edfe5eb9c9d0fe5620","unresolved":true,"context_lines":[{"line_number":108,"context_line":"          \u003cToolbarContent\u003e"},{"line_number":109,"context_line":"            \u003cToolbarItem\u003e"},{"line_number":110,"context_line":"              \u003cspan\u003eShow all pipelines\u003c/span\u003e{\u0027 \u0027}"},{"line_number":111,"context_line":"              \u003cSwitch"},{"line_number":112,"context_line":"                id\u003d\"all-pipeline-switch\""},{"line_number":113,"context_line":"                aria-label\u003d\"Show all pipelines\""},{"line_number":114,"context_line":"                isChecked\u003d{showAllPipelines}"}],"source_content_type":"text/jsx","patch_set":4,"id":"e429a252_81b47048","line":111,"updated":"2024-07-04 13:28:32.000000000","message":"Comment from Tristan on the original change (https://review.opendev.org/c/zuul/zuul/+/916974):\n\u003e I think this needs some padding, after clicking on the button, the border overlap with the label. Perhaps the border should be removed?","commit_id":"ddeccef31a5fbb3d2798f6b361e0b13bcc6322de"},{"author":{"_account_id":27952,"name":"Felix Edel","email":"felix.edel@bmw.de","username":"felix.schmidt"},"change_message_id":"90f299db4c81de25ae951947bd2030de32028c74","unresolved":false,"context_lines":[{"line_number":108,"context_line":"          \u003cToolbarContent\u003e"},{"line_number":109,"context_line":"            \u003cToolbarItem\u003e"},{"line_number":110,"context_line":"              \u003cspan\u003eShow all pipelines\u003c/span\u003e{\u0027 \u0027}"},{"line_number":111,"context_line":"              \u003cSwitch"},{"line_number":112,"context_line":"                id\u003d\"all-pipeline-switch\""},{"line_number":113,"context_line":"                aria-label\u003d\"Show all pipelines\""},{"line_number":114,"context_line":"                isChecked\u003d{showAllPipelines}"}],"source_content_type":"text/jsx","patch_set":4,"id":"82ca049a_1e626a7b","line":111,"in_reply_to":"e429a252_81b47048","updated":"2024-07-09 12:52:21.000000000","message":"Done","commit_id":"ddeccef31a5fbb3d2798f6b361e0b13bcc6322de"}]}