Rodrigo Moya wrote >
>Doug, I left it checked by default because that's the upstream default. I guess it won't harm at all to just have it checked by default for files.
>Also, in your patch, the 2nd gtk_toggle_button_set_active is redundant, since it's already set to FALSE when creating the check button
Actually I just copied that directly from comment 4 in the debian bug page, only changing that one last TRUE to FALSE so the file box wouldn't be enabled (otherwise it would be, though maybe files are set elsewhere?
Anyway I've pretty much never seen complaints about defaults for a filetype being changed so no big deal to be left enabled.
And the whole semantics if the "Remember this.." should really say "Set this application as Default..." is unimportant also
Thanks on behalf of a lot of users for doing a complete fix
Rodrigo Moya wrote > button_ set_active is redundant, since it's already set to FALSE when creating the check button
>Doug, I left it checked by default because that's the upstream default. I guess it won't harm at all to just have it checked by default for files.
>Also, in your patch, the 2nd gtk_toggle_
Actually I just copied that directly from comment 4 in the debian bug page, only changing that one last TRUE to FALSE so the file box wouldn't be enabled (otherwise it would be, though maybe files are set elsewhere?
Anyway I've pretty much never seen complaints about defaults for a filetype being changed so no big deal to be left enabled.
And the whole semantics if the "Remember this.." should really say "Set this application as Default..." is unimportant also
Thanks on behalf of a lot of users for doing a complete fix