Am 18.07.2010 um 11:44 schrieb Pavel Sanda:

> Stephan Witt wrote:
>>                                      scroll_whell_zoom = 
>> SCROLL_WHEEL_ZOOM_SHIFT;
>> -                            else if (tmp == "option")
>> -                                    scroll_whell_zoom = 
>> SCROLL_WHEEL_ZOOM_OPTION;
>> +                            else if (tmp == "alt")
>> +                                    scroll_whell_zoom = 
>> SCROLL_WHEEL_ZOOM_ALT;
>>                              else {
>>                                      scroll_whell_zoom = 
>> SCROLL_WHEEL_ZOOM_OFF;
>>                                      if (tmp != "off" && tmp != "false")
>> @@ -2604,8 +2604,8 @@
>>                      case SCROLL_WHEEL_ZOOM_SHIFT:
>>                              status = "shift";
>>                              break;
>> -                    case SCROLL_WHEEL_ZOOM_OPTION:
>> -                            status = "option";
>> +                    case SCROLL_WHEEL_ZOOM_ALT:
>> +                            status = "alt";
> 
> maybe
> 
> ifdef mac
> status="option";
> endif
> 
> to make everybody happy?

My keyboard has that key labeled with "alt" too. So "alt" is valid too. (I as 
german speaker associate it with "old"... so I don't like it so much - but 
that's not important).

This change is part one of the solution and platform independent. I'll commit 
the patch above now.
The next step is to make the Mac specific changes. I'll prepare a patch and 
post it soon. But I can wait with that after alpha5.

The more serious problem I see is that the recent changes with CMake broke the 
"Out-of-the-Box" build here. The deactivate of Aspell per default did not 
removed the "src/AspellChecker.cpp" file from the list off targets and the 
build failed. Maybe, this should be fixed before alpha5? Peter, is it really 
working for you?

Stephan


Reply via email to