[Chandler-dev] RenameAndNewEvent fails on Mac

[D John Anderson]

Hi Philippe:

Did you fix the mini calendar size so that it shows only one month? I  
think that is the correct fix for this problem. I suspect that this  
problem was caused by the fact that the new collection was partially  
obscured by the minicalendar, not that the UI changed in a way that  
made old mouse positions invalid.

I don't think the script recording depends too much on mouse position  
-- most of the events don't use mouse position, even though the  
position is recorded and you may see it in the script.  There may be  
a few exceptions, but they are usually easy to fix. For example, we  
can easily use row number instead mouse location for selecting items  
in tables (e.g. the sidebar).

John

On Jul 6, 2007, at 11:51 PM, Philippe Bossut wrote:

> Hi,
>
> I arrived at the same conclusion on my side. I checked a fix in  
> r14940. Note that recTestNewEvent had the same problem.
>
> Now, the real issue is that the script recording is sensitive to  
> rendering condition which means my tweaks (on Mac) do not work on  
> Linux: this is because the position of the divider between mini cal  
> and collections depends on the font size used for the various text  
> (in the day preview and the minical) and that is not consistent  
> from a platform to the next. This also really means that a  
> functional test framework based on the capture of mouse coordinates  
> can't  work crossplatform.
>
> How do we solve that?
>
> - Philippe
>
> D John Anderson wrote:
>> Hi:
>>
>> Dan and I just tracked down the cause of RenameAndNewEvent failing  
>> on Mac. A recent change (I'm not sure of the revision) causes the  
>> Mini calendar to be too large -- which makes the sidebar smaller  
>> (not so tall) and the test depends on a sidebar with enough room  
>> to edit a new collection that is completely visible.
>>
>> You can launch Chandler, make the mini calendar smaller and the  
>> test passes. Also the failure it detects is consistent with that  
>> change.
>>
>> Dan said the window was recently made smaller and that's probably  
>> the problem. If the window gets smaller the mini cal should  
>> probably only show one month instead of two.
>>
>> John
>> _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _
>>
>> Open Source Applications Foundation "chandler-dev" mailing list
>> http://lists.osafoundation.org/mailman/listinfo/chandler-dev