oops, forgot to hit the publish button, should be up now
http://reviewboard.kde.org/r/1248/
On Sun, Aug 9, 2009 at 3:00 PM, Patrick Aljord wrote:
> http://reviewboard.kde.org/r/1248/ here is the diff with the
> imagebinPrivacy always as int.
>
___
Plasma
http://reviewboard.kde.org/r/1248/ here is the diff with the
imagebinPrivacy always as int.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
On August 9, 2009 12:42:30 Patrick Aljord wrote:
> On Sat, Aug 8, 2009 at 1:56 PM, Chani wrote:
> > kompare won't load the diff for some reason...
>
> I used the techbase tutorial here
> http://techbase.kde.org/Contribute/Send_Patches#Creating_a_Simple_File_Patc
>h and used that command:
>
> svn d
On Sat, Aug 8, 2009 at 1:56 PM, Chani wrote:
>
> kompare won't load the diff for some reason...
I used the techbase tutorial here
http://techbase.kde.org/Contribute/Send_Patches#Creating_a_Simple_File_Patch
and used that command:
svn diff --diff-cmd diff --extensions "-u -p"
I was thinking with time we could add more options for alll the
different servers, then a new page would make sense, for now it's true
that it looks empty.
___
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-dev
On August 8, 2009 18:41:32 Patrick Aljord wrote:
> Thanks to all for your comments. I haven't coded anything in C++ for
> at least 3 years so I guess it shows :)
>
> I'll send the patch with your suggestions tomorrow, in the meantime
> here is the screenshot http://imagebin.ca/view/fyaoR7T.html
hm
Thanks to all for your comments. I haven't coded anything in C++ for
at least 3 years so I guess it shows :)
I'll send the patch with your suggestions tomorrow, in the meantime
here is the screenshot http://imagebin.ca/view/fyaoR7T.html
___
Plasma-devel
On Saturday 08 August 2009, 16:35 Chani wrote:
> oh.
> but we also have this:
> int imagebinPrivacy = uiServersOptions.imagebinPrivacy->currentIndex();
>
> so, either make it always an int, or make it an enum. don't use a string.
> especially a string that just contains a number anyways...
+1 for
> > one thing I don't get is how you managed to use a string ("0") as the
> > default value for an int ;)
>
> +QString m_privacy;
>
> m_privacy seems to be a QString :)
oh.
but we also have this:
int imagebinPrivacy = uiServersOptions.imagebinPrivacy->currentIndex();
so, either make it alway
On Saturday 08 August 2009, 13:40 Patrick Aljord wrote:
> This is my first patch, it just adds "posting private picture to
> imagebin.ca from the pastebin plasmoid" as the title says, it needs to
> be applied from kdeplasma-addons, hope it works for you.
Thanks for the patch! As Chani asked, can y
On August 8, 2009 09:40:28 Patrick Aljord wrote:
> Hey all,
>
> This is my first patch, it just adds "posting private picture to
> imagebin.ca from the pastebin plasmoid" as the title says, it needs to
> be applied from kdeplasma-addons, hope it works for you.
>
> Cheers,
>
> Pat
thanks for the pa
Hey all,
This is my first patch, it just adds "posting private picture to
imagebin.ca from the pastebin plasmoid" as the title says, it needs to
be applied from kdeplasma-addons, hope it works for you.
Cheers,
Pat
Index: applets/pastebin/pastebin.cpp
=
12 matches
Mail list logo
