Zim

Merge lp://staging/~o-scholder/zim/pyzim-refactor into lp://staging/~jaap.karssenberg/zim/pyzim-refactor

Proposed by Olivier Scholder
Status: Needs review
Proposed branch: lp://staging/~o-scholder/zim/pyzim-refactor
Merge into: lp://staging/~jaap.karssenberg/zim/pyzim-refactor
Diff against target: 65 lines (+26/-1)
3 files modified
data/manual/Plugins/GNU_R_Plot_Editor.txt (+11/-0)
data/templates/plugins/gnu_r_editor.r (+1/-1)
zim/plugins/gnu_r_ploteditor.py (+14/-0)
To merge this branch: bzr merge lp://staging/~o-scholder/zim/pyzim-refactor
Reviewer Review Type Date Requested Status
Jaap Karssenberg Pending
Review via email: mp+197506@code.staging.launchpad.net

Description of the change

Hi Jaap,
Yes my mistake. Sorry. I hope I merge into the right branch now.

I hope you think the tweak is useful. I myself often need wide plots and needed it absolutely.

regards

Olivier

To post a comment you must log in.
Revision history for this message
Jaap Karssenberg (jaap.karssenberg) wrote :

Looks very useful indeed - will merge it. Thanks!

Only wondering about the default - maybe it make sense to not set a default (or make it a preference) and without default simply let R decide the size ?

Btw. thinking about a way to automatically scale all images to window size in the page view, but will be later improvement.

Revision history for this message
Olivier Scholder (o-scholder) wrote :

according to:
http://stat.ethz.ch/R-manual/R-devel/library/grDevices/html/png.html
480px is the default size for R.

I guess taking the 480px from the preference is for sure a good idea. So
that everybody can set to the value they like and work with it.

For the resizing, the problem is that each images are not created
on-the-fly by zim. They are converted into picture only once they are
edited. This will makes the auto-resizing to the screen size difficult. We
can implement that in the image module, but the problem with plots is that
they are usually made for a specific width/height ratio and thus automatic
resizing might become annoying.

For the image we have in the source (the saved txt files):

{{PATH-to-myimage.png?width=320}}

we can easily let the user write something like:

{{PATH-to-myimage.png?width=50%}}

and calculate the size of the image on the fly.

what do you think?

2013/12/3 Jaap Karssenberg <email address hidden>

> Looks very useful indeed - will merge it. Thanks!
>
> Only wondering about the default - maybe it make sense to not set a
> default (or make it a preference) and without default simply let R decide
> the size ?
>
> Btw. thinking about a way to automatically scale all images to window size
> in the page view, but will be later improvement.
> --
> https://code.launchpad.net/~o-scholder/zim/pyzim-refactor/+merge/197506
> You are the owner of lp:~o-scholder/zim/pyzim-refactor.
>

Unmerged revisions

683. By Olivier Scholder

Add size tweak for GNU-R plugin

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches