> I didn't review all the code changes in detail but tested it and works fine 
> for me.
> 
> I wonder if we could make it visually more obvious that the directory and 
> interval options belongs to the respective radio options. Maybe indenting 
> them or wrap them in a `GtkFrame`?

i tried indenting blocks initially but rolled back because increase of overall 
height for this tab increases height for all of them. 
Both indentation and frame wrapping does not look ideal to me, but if we want 
we may use one or another.
I committed indentation for now

![image](https://github.com/user-attachments/assets/9eabee61-5a5b-4627-b6d2-696e47ac6a97)

![image](https://github.com/user-attachments/assets/6040a84a-f489-4d12-8cc1-45d96a370803)


-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3911#issuecomment-2486595515
You are receiving this because you are subscribed to this thread.

Message ID: <geany/geany/pull/3911/[email protected]>

Reply via email to