Closing the last few issues

Henrik Brautaset Aronsen subsurface at henrik.synth.no
Tue Feb 12 04:05:37 PST 2013


I submitted your patch without the satellite.* stuff and added your text 
as comment -- in addition to my macos patch.

Now, if Lubomir could add a windows patch on top of that... :)

Henrik

Henrik Brautaset Aronsen wrote:
> Amit, you need to remove the satellite.* stuff from your patches :)
>
> H
>
> Amit Chaudhuri wrote:
>> Hi..
>>
>> Attached is a second attempt at the window save functionality.  It adds
>> simple checks for a recollection of a large screen from a previous
>> session and either sets geometry to 70% (arbitrary choice) of currency
>> screen dims, starting at (0,0),  or to the remembered dims.
>>
>> I did consider opting for a "maximise" if previous screen too large for
>> current, but figured some people might really hate that (but I might be
>> wrong?).
>>
>> I have not (yet) looked at replacing the <>..set_conf_int interface with
>> reuse of the string equivalent suggested by Dirk.  I'm not sure I'll get
>> to that on V3.0 timescales.  If anyone else can - I'm fine with that.
>>
>> That means for full support this would need the previous contribution
>> from Henrik (OSX), plus same for Windows (don't think I saw anything on
>> that front).
>>
>> I realise that this doesn't address all concerns flagged previously, but
>> the "big screen last time checks" seemed to me to have the greatest
>> impact on the user.  Happy to re-visit the behind the scenes stuff for
>> 3.1, or for someone else to add string parsing stuff for 3.0.
>>
>> Signed off  by Amit Chaudhuri <amit.k.chaudhuri at gmail.com
>> <mailto:amit.k.chaudhuri at gmail.com>>
>>
>> Prepared against master this morning.  If I click on master in gitk this
>> is what I get as SHA1: f50ae3a6897d35245f7679eddf6d7f820aecd235
>>
>> Off the the shops to buy a new (colourful) shirt....
>>
>> A
>>
>> _______________________________________________
>> subsurface mailing list
>> subsurface at hohndel.org
>> http://lists.hohndel.org/cgi-bin/mailman/listinfo/subsurface


More information about the subsurface mailing list