Pavel Sanda wrote:
> kuem...@lyx.org wrote:
>> +struct LevelButton : QRadioButton
>> +{
>> +    LevelButton(const QString& name) : QRadioButton(name) {}
>> +    Debug::Type level;
>> +};
> 
> we will need checkboxes or buttons, since the levels can be combined.
> you can set eg files+version control debug level.
> 
> i'm not sure its easy job, it would be fine to use use more horizontal space 
> of the window
> for the boxes, eg. two or three columns.

feel free to visit your friend 'designer' ;)
But two columns shouldn't be too hard.

> 
>> +void GuiProgressView::levelChanged(QAbstractButton* b)
>> +{
>> +    LevelButton* lb = dynamic_cast<LevelButton*>(b);
>> +    if (lb)
>> +            lyxerr.level(lb->level);
> 
> i crafted special lfun for this since i think we shouldn't set such things
> directly from ui.

but it needs a string.

> 
>> @@ -19,7 +20,7 @@
>>  #define LYXDEBUG_H
>>  
>>  #include "support/strfwd.h"
>> -
>> +#include <vector>
> 
> ouch ouch ouch. we have to avoid this since debug.h is included everywhere and
> we have to keep it as as tiny as possible.

is gone now

> 
> if you find time to fix those, i'll be happy. if not i fix them myself
> but it will take some time, i'm not on my box next days :)
> 
> bye
> pavel
> 

Reply via email to