Code review comment for lp:~mterry/indicator-me/gsettings

Revision history for this message
Michael Terry (mterry) wrote :

I agree that this key would be an enum if we designed it from scratch. However, the gsettings documentation leads me to believe that enums are stored as strings. And gsettings-data-convert does not handle any sort of data transformation.

So you'd have to use some new key like 'display_enum', keep 'display' around, and then convert values yourself after both are in dconf. Doesn't seem worth it.

« Back to merge proposal