Ubuntu
ipe package

More default colors should be provided (simple patch)

Bug #162213 reported by Chris Conway on 2007-11-12

Affects		Status	Importance	Assigned to	Milestone
	ipe (Debian)	Fix Released	Unknown	debbugs #451022
	ipe (Ubuntu)	Fix Released	Wishlist	Unassigned

Bug Description

Binary package hint: ipe

The upstream developer purposefully cripples the color selection: "The standard style sheet is deliberately short, to encourage users to go out and learn how to make their own." (See here: http://tclab.kaist.ac.kr/ipe/index_1.html) Personally, I think this is a bad policy, and the default installation should at least include cyan, magenta, orange, and other common colors. Better, it should include a selection of colors competitive with what, e.g., OpenOffice.org provides.

This is a simple patch to src/ipelib/ipestdstyles.cpp that should be very easy to maintain. I'll even volunteer to maintain it. I'll upload a proposed patch later.

Related branches

lp:ubuntu/karmic/ipe

Andrea Colangelo (warp10) on 2007-11-12

Changed in ipe:
assignee:	nobody → warp10
status:	New → In Progress

Cesare Tirabassi (norsetto) on 2007-11-12

Changed in ipe:
importance:	Undecided → Wishlist

Revision history for this message

Stephan Rügamer (sruegamer) wrote on 2007-11-12:

Hi Chris, Andrea,

I'm working right now on the ipe merge (I fixed some things the last time).
I'll provide some default colours like OOFice..

Add the patch to the merge bug.

Regards,

\sh

Revision history for this message

Chris Conway (cconway) wrote on 2007-11-12:

You mean #161191?

Revision history for this message

Stephan Rügamer (sruegamer) wrote on 2007-11-12:

hi Chris,

right, it's https://bugs.edge.launchpad.net/ubuntu/+source/ipe/+bug/161191

I Included now all colors from colors.xml which is delivered in /usr/share/doc/ipe/doc/colors.xml.gz

Regards,

\sh

Revision history for this message

Chris Conway (cconway) wrote on 2007-11-12: Re: [Bug 162213] Re: More default colors should be provided (simple patch)

On Nov 12, 2007 3:09 PM, Stephan Hermann <email address hidden> wrote:
> I Included now all colors from colors.xml which is delivered in
> /usr/share/doc/ipe/doc/colors.xml.gz

I don't recommend that approach. To see why, Ctrl-Right click on an
entity to bring up a context menu, then hover over stroke or fill: the
pop-up menu fills the screen with over 600 color choices. Attached is
a pared down list of about 30 common colors that I'm now using as my
defaults. (An alternative, of course, would be to wrap the color list
in a scrolling pop-up menu control, but that's a more complicated
patch.)

Thanks,
Chris

Andrea Colangelo (warp10) on 2007-11-12

Changed in ipe:
assignee:	warp10 → nobody
status:	In Progress → Confirmed

Revision history for this message

Cesare Tirabassi (norsetto) wrote on 2007-11-12:

Attachments are lost if you are sending an email. Please attach it to the bug report. Thanks.

Revision history for this message

Chris Conway (cconway) wrote on 2007-11-12:

Pared-down color settings Edit (1.2 KiB, text/xml)

Sorry about that.

Revision history for this message

Stephan Rügamer (sruegamer) wrote on 2007-11-12:

Chris,

what about a plain 16 color palette for it?
The good thing is, a complete color style is provided in the docs directory.
So the user just adds it and good.

I agree here, that 4 or 5 colors are just to less for a professional application.

Regards,

\sh

Cesare Tirabassi (norsetto) on 2007-11-12

Changed in ipe:
assignee:	nobody → norsetto
status:	Confirmed → In Progress

Bug Watch Updater (bug-watch-updater) on 2007-11-13

Changed in ipe:
status:	Unknown → New

Cesare Tirabassi (norsetto) on 2007-11-19

Changed in ipe:
assignee:	norsetto → nobody
status:	In Progress → Fix Committed

Revision history for this message

Stephan Rügamer (sruegamer) wrote on 2007-11-19:

ipe (6.0pre28-4ubuntu1) hardy; urgency=low

  * Merge from debian unstable, remaining changes: (LP: #161191)
    + debian/ipe.{desktop,png.uu}:
      - added files to provide an icon and desktop file
    + debian/control:
      - Updated to match Ubuntu Maintainer Policy
      - Added Build-Dep for sharutils
    + debian/rules:
      - Added lines to install desktop and png file.
  * debian/ipe.desktop:
    - removed deprecated "Encoding" key
    - reworded Comment key in line with HIG
  * src/ipelib/ipestdstyles.cpp: (LP: #162213)
    - added colors to default stylesheet (palette courtesy of Chris Conway).

ipe (6.0pre28-4) unstable; urgency=low

* debian/control: Update depends and conflicts to new freetype (version
2.3.5). Closes: #432476.

  * debian/control: Add (<< 2007) to dependency on texlive-latex-base,
    since ipe can't cope with the output of version 1.40 of Pdflatex
    (included in in TexLive 2007). Closes: #432043.

* Update homepage URL. Closes: #427388.

  * src/config.pri: Remove -Werror as upstream suggests
    (http://s105.theano.de/cgi-bin/bugzilla/show_bug.cgi?id=227). Closes:
    #417675.

-- Stephan Hermann <email address hidden> Sat, 10 Nov 2007 14:31:25 +0100