Xournal for Fremantle - Page 27 - maemo.org

anidel | # 261 | 2010-02-07, 20:58 | Report

Be a little more patient

sorry about that..

Cool video zehjotkah, you also discovered a known (to me at least :P) bug in Xournal (Save As uses / as default storage directory instead of Documents, when you hit Save, it tried to store in / and you can't.)

That's already been fixed 'internally'.

Aniello

The Following 2 Users Say Thank You to anidel For This Useful Post:
iKneaDough, zehjotkah

mclarson | # 262 | 2010-02-07, 22:06 | Report

I really loved this program on the n810 and I'm really glad it was updated for n900 so quickly. A couple features I *really* miss are:

1. PDF annotation.
2. Select rectangle, to move stuff around.

These 2 feature I used to use *alot*. Drawing seems smoother in this version though.

Are these features planned for inclusion?

anidel | # 263 | 2010-02-07, 22:42 | Report

PDF annotation is there.. just open a PDF file.

About 2:

all that tools that were there, are still there.. just missing an UI element to activate it.
All are coming back with the next release....

Again.. a little patience...
Aniello

The Following 6 Users Say Thank You to anidel For This Useful Post:
codeMonkey, debernardis, iKneaDough, mclarson, qole, Shanezlar

anidel | # 264 | 2010-02-08, 16:09 | Report

As an update of what you'll see next (need to update first post).

Anyway.. I'll release a new version with what came up from Barcelona.
This will bring many old features that are absent from Xournal for Fremantle.

Once this is done, I'll update my code to Xournal 0.4.5 and try to get my patches integrated into the main Denis code and put the code into GIT so that people can contribute with patches and I can track versions easily (and fix version's specific bugs).

After that I'd like to integrate a few cool patches (like search and hrw).
Once this is done.. the Next Gen UI will be my main focus.

We'll see how it goes.. this is life in the open source world

things change and even small events can significantly alter the roadmap.

Aniello

The Following 7 Users Say Thank You to anidel For This Useful Post:
codeMonkey, debernardis, iKneaDough, Maj3stic, ofels, qole

edanaher | # 265 | 2010-02-10, 17:39 | Report

Wow... I was looking for a way to grade papers electronically, and this fits the bill - I can draw on student's papers and type comments. Unfortunately, when I try to save as a pdf after typing in text, it segfaults on saving and writes a truncated pdf... it looks just like the the bug reported here:

https://bugs.launchpad.net/ubuntu/+s...al/+bug/474468

However, the fix reported in comment #4 (changing a gsize to a guint32) doesn't seem to work here.

Am I the only one having this problem? It's very consistent: I can open up a pdf, draw on it and export it with no problem, but as soon as I add text, exporting it as a pdf crashes.

(To get the text tool, I tweaked the "HildonToolBar" section of /usr/share/xournal/xo-hildon-ui.xml to replace "Highlighter" with "Text")

anidel | # 266 | 2010-02-10, 18:08 | Report

When you save it, do you save it in Documents or in "/" ?

edanaher | # 267 | 2010-02-10, 18:31 | Report

Originally Posted by anidel View Post

When you save it, do you save it in Documents or in "/" ?

I tried both, and both crash. Similarly, saving in either works if I didn't add any text to the document. (Where I'm interpreting "/" as {devicename}/, since I am unable to select "/" as the directory to save.

I'll poke around with it a bit more tonight and see if I can give you more useful information.

edanaher | # 268 | 2010-02-10, 23:24 | Report

So, line 858 or so of xo-print.c (I may have messed up the line numbers very slightly with my debugging stuff) is

Code:

tmpfile = mkstemp(g_strdup(TMPDIR_TEMPLATE));

tmpfile is a char*, mkstemp returns a file handle. If, instead, you use mktemp, it gives you the file name, which is then what the later functions expect.

Kind of a silly mistake... not sure where it came from, but easy to fix. I can now export pdf files with text it them, and it'll get some good stress-testing over the next week with grading.

anidel | # 269 | 2010-02-10, 23:30 | Report

Originally Posted by edanaher View Post

So, line 858 or so of xo-print.c (I may have messed up the line numbers very slightly with my debugging stuff) is

Code:

tmpfile = mkstemp(g_strdup(TMPDIR_TEMPLATE));

tmpfile is a char*, mkstemp returns a file handle. If, instead, you use mktemp, it gives you the file name, which is then what the later functions expect.

Kind of a silly mistake... not sure where it came from, but easy to fix. I can now export pdf files with text it them, and it'll get some good stress-testing over the next week with grading.

Are you sure about that?
In my code here there is no mkstemp, but mktemp...

But anyway, I'm syncing my code with upstream 0.4.5 and Denis swtched to libpoppler and completely rewrote that function.

But weird you found this mkstemp call...

Aniello

anidel | # 270 | 2010-02-12, 01:20 | Report

Just to let you guys know that I have "internally" (

) updated Xournal to mainstream 0.4.5

This fixes above issue with text as well, removes libgnomeprint dependancy (depends now on poppler) and will be named 0.4.5-1fremantle1 (a fresh start).

Should also come with more translations.

Hope to be able to release it asap.

Aniello

The Following 4 Users Say Thank You to anidel For This Useful Post:
BruceL, codeMonkey, debernardis