Adit Panchal
2005-03-05 08:48:02 UTC
I know this may be unnecessary and some people might think "What's the
point, if you just check your settings, you can find it out?" But I
think it is rather helpful to know what phone model I am currently
using. I switch between two phones and seeing the phone model in the
status bar makes it easier to know whether I need to get in the
preferences to change it.
It would probably also cut down on questions on bitpim-user that relate
to not changing the default of the LG-VX4400 to the right phone model.
Attached is a diff that should enable it. The only thing is that I
added two lines when the config is updated in guihelper that will
update the statusbar after a model change. I think a better way of
getting the phone model is reading it out of pubsub. Unfortunately, it
is late for me, and I can't figure out which piece of code subscribes
to PHONE_MODEL_CHANGED. If I can figure that out, I can redo it so that
each time that particular subscriber calls PHONE_MODEL_CHANGED, it will
update the status bar. For now though, the patch works without error.
Thanks,
Adit
point, if you just check your settings, you can find it out?" But I
think it is rather helpful to know what phone model I am currently
using. I switch between two phones and seeing the phone model in the
status bar makes it easier to know whether I need to get in the
preferences to change it.
It would probably also cut down on questions on bitpim-user that relate
to not changing the default of the LG-VX4400 to the right phone model.
Attached is a diff that should enable it. The only thing is that I
added two lines when the config is updated in guihelper that will
update the statusbar after a model change. I think a better way of
getting the phone model is reading it out of pubsub. Unfortunately, it
is late for me, and I can't figure out which piece of code subscribes
to PHONE_MODEL_CHANGED. If I can figure that out, I can redo it so that
each time that particular subscriber calls PHONE_MODEL_CHANGED, it will
update the status bar. For now though, the patch works without error.
Thanks,
Adit