Re: [Inkscape-devel] OCAL Dialog Redesign
Hi Guys,
So I have been doing a lot of work on the Import from OCAL dialog and I have finally finished! I was wondering whether I could get some testing, especially on some different platforms with different internet connections.
All you need to do is get my branch using: bzr branch lp:~and471/inkscape/ocal-dialog-improvements
And then compile as usual. You will find the dialog in File > Import Clip Art
So yeah, basically I want a lot of testing, to iron out any errors that I have missed and also feedback on the stuff I have done. Then once that is all done, I shall propose for merging into trunk.
A list of changes is below, as well as a link to screenshots of the new dialog.
Thanks!
Hey,
Would it be possible for you to pull your branch up-to-date with trunk (if it isn't already), and then create a patch and then upload it and link it from that wiki page? (more people are likely to test that way than another checkout)
Cheers, Josh
On Mon, Apr 4, 2011 at 9:03 AM, Andrew <rugby471@...400...> wrote:
Hi Guys,
So I have been doing a lot of work on the Import from OCAL dialog and I have finally finished! I was wondering whether I could get some testing, especially on some different platforms with different internet connections.
All you need to do is get my branch using: bzr branch lp:~and471/inkscape/ocal-dialog-improvements
And then compile as usual. You will find the dialog in File > Import Clip Art
So yeah, basically I want a lot of testing, to iron out any errors that I have missed and also feedback on the stuff I have done. Then once that is all done, I shall propose for merging into trunk.
A list of changes is below, as well as a link to screenshots of the new dialog.
Thanks!
-- Andrew
Screenshots: http://wiki.inkscape.org/wiki/index.php/OCAL_Dialog_Re-Design
== Changes ==
UI Redesign (which includes)
- Altered the padding and packing of all widgets
- Added search entry with clear and search icons
- Added OCAL Logo overlay
- Put descriptions inline in list
- New preview widget that shows extra metadata
- New status widget to notify on errors/success etc.
- Dialog is now persistent (retains state)
- Better messages for the user
- Changed title/menuitem to 'Import Clip Art'
Backend stuff (which includes)
- Use new GFile API instead of depreciated GNOMEVFS
- Use async methods so the UI isn't blocked
- Use smaller, thumbnail images in the preview widget (save on
bandwidth and increase speed) - Implement cancelling of operations - Put downloaded resources into one directory, from which we can use later - Big cleanup in terms of code
Create and publish websites with WebMatrix Use the most popular FREE web apps or write code yourself; WebMatrix provides all the features you need to develop and publish your website. http://p.sf.net/sfu/ms-webmatrix-sf _______________________________________________ Inkscape-devel mailing list Inkscape-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/inkscape-devel
Hi,
De : Josh Andler <scislac@...400...> Would it be possible for you to pull your branch up-to-date with trunk (if it isn't already), and then create a patch and then upload it and link it from that
wiki page? (more people are likely to test that way than another checkout)
I've just set a branch merge proposal. A branch diff will be available on the branch page shortly (https://code.launchpad.net/~and471/inkscape/ocal-dialog-improvements/+merge/...).
Regards. -- Nicolas
Hi,
So I have kind of hit a road block in the OCAL stuff. Basically su-v informed me that the method I used to download the XML file (to obtain search results), Gio::File::load_contents_async doesn't actually work on Mac OSX.
Therefore I would like to change this to read_async, however I don't really have any experience with it and as a result, it isn't actually working :/
Does anyone have any experience? And if so, could they look at the code in src/ui/dialog/ocaldialogs.cpp lines 949-1080
Thanks
On Apr 12, 2011, at 11:32 PM, Andrew wrote:
Hi,
So I have kind of hit a road block in the OCAL stuff. Basically su-v informed me that the method I used to download the XML file (to obtain search results), Gio::File::load_contents_async doesn't actually work on Mac OSX.
Therefore I would like to change this to read_async, however I don't really have any experience with it and as a result, it isn't actually working :/
Does anyone have any experience? And if so, could they look at the code in src/ui/dialog/ocaldialogs.cpp lines 949-1080
My Inkscape time is freeing up again finally, so I can help you poke about a bit.
On 13/04/2011 08:32, Andrew wrote:
So I have kind of hit a road block in the OCAL stuff. Basically su-v informed me that the method I used to download the XML file (to obtain search results), Gio::File::load_contents_async doesn't actually work on Mac OSX.
Therefore I would like to change this to read_async, however I don't really have any experience with it and as a result, it isn't actually working :/
Does anyone have any experience? And if so, could they look at the code in src/ui/dialog/ocaldialogs.cpp lines 949-1080
*bump*
Andrew has provided a update for the merge proposal:
On 27/12/2011 22:10, Andrew wrote:
Hi, sorry for my long absense form this but I have just fixed the issue that JazzyNico reported above.
The branch is ready for merging into trunk. Please someone do it quickly! :)
from https://code.launchpad.net/~and471/inkscape/ocal-dialog-improvements/+merge/56234
but it still fails on osx (tested this time on OS X 10.7.2 Lion (Xcode 4.2.1), with dependencies via up-to-date MacPorts, using llvm-gcc-4.2.1 as compiler). [1]
Merging as is will remove a currently working feature from the osx port of Inkscape. On the other hand the new dialog seems to work in the Windows port of Inkscape (which doesn't support importing from OpenClipart in current stable and trunk).
Trading one supported platform for another one (with certainly more users)? Or can anyone help with the changes mentioned by Andrew (using 'read_async' instead of 'load_contents_async', see above)? Or would it be feasible to keep the old code in place to be used when configuring/building on darwin/osx (until a working solution in Inkscape or upstream GLib/GIO is found)?
~suv
[1] This could well be another upstream issue, similar to the GIO issue with the 'Gallery' dialog proposal discussed earlier: http://thread.gmane.org/gmane.comp.graphics.inkscape.devel/36401/focus=36437
participants (5)
-
Andrew -
Jon Cruz -
Josh Andler -
Nicolas Dufour -
~suv