Hello!

Enguerrand de Rochefort has written on Thursday, 19 January, at  0:18:

>I am working on a patch for lxpanel that allows to configure taskbars such 
>that they only show tasks on a specific workspace. That way, one can have one 
>taskbar for each workspace and have it display all tasks on that (and only 
>that) workspace.
>The behavior is similar to what the tint2 bar does.

Well, taskbar has an option to show windows from only current desktop. Do
you mean your change is not for current one but for fixed one, right?

>At this point I got the functionality working but it must be configured by 
>manually editing the configuration file.

>What's missing is the extension of the configuration dialog, which I probably 
>wouldn't bother to implement for myself but would be willing to take a crack 
>at if you guys were interested to merge the change. (I have very little 
>experience in c programming so do expect this to take a while...)

It is not a big problem, it's what is collective development for - you
miss some element of feature, someone else adds/fixes it.

>Before doing that (and creating a sourceforge account, or what's your 
>preferred way of contributing code?) I just wanted to get some initial 
>feedback on this.
>You can find my current diff below (created with git format-patch).

Sourceforge account isn't required for a contribution, you may send a
patch here (as you did now), and request a git account later, after other
developers see your changes are sane enough.

>Let me know what you think,
>kind regards

Overall your change seems OK. Although I would rather name the field like
'fixed_desktop_num' with value -1 to do current behavior (show all) and
working only if 'show_all_desks' is TRUE, to keep compatibility.

>Enguerrand

    With best regards,
    Andriy.

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, SlashDot.org! http://sdm.link/slashdot
_______________________________________________
Lxde-list mailing list
Lxde-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/lxde-list

Reply via email to