)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":37598,"name":"Ivan Anfimov","display_name":"Ivan Anfimov","email":"lazekteam@gmail.com","username":"anfimovir"},"change_message_id":"0bb3f5ac398396027a08ac9a6433b4ad651d3a5c","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"70bf4055_1556440d","updated":"2026-07-21 03:46:46.000000000","message":"@ali74.ebrahimpour@gmail.com\n\nHello Ali,\n\nFirst of all, I would like to say a big thank you for your participation in the development of Horizon and your contribution.\n\nThe idea of ​​a dark theme is truly relevant and supports modern trends in web development, but I would like to draw special attention to the way you have implemented it now - simply creating a separate theme is not bad for local use, but from a development perspective, it will add additional effort to its support in the event of any changes and testing.\n\nA more reliable way in this case would be to refine the main theme, something like this:\n\n\u003cstyle\u003e\n:root {\n    --background: #fff;\n    --secondary: #666;\n    --foreground: #000\n}\n\n@media (prefers-color-scheme: dark) {\n    :root {\n        --background: #000;\n        --secondary: #A1A1A1;\n        --foreground: #fff\n    }\n}\n\nhtml, body {\n    overflow: hidden;\n    background: var(--background);\n    color:var(--foreground)\n}\n\n*, ::after, ::before {\n    box-sizing: border-box;\n    margin:0\n}\n\nAnd etc.. And adding switches to UI to switch between light and dark themes, or the system mode. This will also help prevent future duplication of the same theme.","commit_id":"a356268c326e5f040bc8f2871764de2eec9955fa"},{"author":{"_account_id":35133,"name":"Jan Jasek","email":"jjasek@redhat.com","username":"janjasek"},"change_message_id":"f27b1fa53f191221b7a64f394b2556e2a9240197","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d01a17c7_80695f2c","updated":"2026-07-20 12:41:15.000000000","message":"recheck","commit_id":"a356268c326e5f040bc8f2871764de2eec9955fa"},{"author":{"_account_id":35595,"name":"ali","email":"ali74.ebrahimpour@gmail.com","username":"ali"},"change_message_id":"64dc730318864caab953765185bc23eb5f2a5b96","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"3ff5e185_5992415d","in_reply_to":"70bf4055_1556440d","updated":"2026-07-21 12:32:45.000000000","message":"Hello,\n\nThank you for your detailed feedback and for taking the time to review this contribution.\n\nI completely agree that maintaining two separate themes can introduce additional overhead and duplication in the long term. The current implementation was mainly focused on validating the dark theme integration and providing a working prototype for Horizon.\n\nI agree that the better approach would be to move toward a theme system based on CSS variables (design tokens), where the main theme defines shared variables and light/dark modes only override those values. This would make future changes easier and avoid maintaining duplicated styles.\n\nI will revise the implementation by:\n\n* introducing CSS variables for common colors and UI elements,\n* adding dark mode support based on those variables,\n* providing a UI switch for light/dark/system preference,\n* ensuring existing Horizon styles continue to work without duplicated theme files.\n\nI appreciate the suggestion and will update the implementation accordingly. I believe this approach will make dark mode a more maintainable and native feature for Horizon rather than a separate theme.\nThank you again for the review.","commit_id":"a356268c326e5f040bc8f2871764de2eec9955fa"}]}