Opened 15 years ago

Closed 13 years ago

Last modified 11 years ago

#1458 closed enhancement (fixed)

Write - export as PDF

Reported by: iwikiwi Owned by: godiard
Priority: Unspecified by Maintainer Milestone:
Component: Write Version: Unspecified
Severity: Minor Keywords: r!
Cc: sascha_silbe Distribution/OS: Unspecified
Bug Status: Assigned


So right now even though the API is enough exposed to allow Write to export a document as a pdf, it can't, as we are not making use of the capability.
As part of the demo at the iitm hackfest, we have created a small patch which corrects this.

Attachments (2)

pdfwrite.patch (853 bytes) - added by iwikiwi 15 years ago.
Patch for this enhancement.
0007-Add-the-option-to-export-to-PDF-SL-1458.patch (982 bytes) - added by godiard 14 years ago.

Download all attachments as: .zip

Change History (11)

Changed 15 years ago by iwikiwi

Patch for this enhancement.

comment:1 Changed 15 years ago by sascha_silbe

  • Bug Status changed from Resolved to Assigned
  • Cc sascha_silbe added; tomeu removed
  • Distribution/OS changed from Fedora to Unspecified
  • Keywords r? added; Write enhancement removed
  • Priority changed from High to Unspecified by Maintainer

First of all, thanks for the patch, it's most welcome!

Please see Code Review for an explanation of how to get your patch into the official sources. I've already fixed up the fields of this ticket to make it enter the review queue.
For future patches the Code Guidelines might be good to know about as well, but for the current one everything is fine.

The patch is rather small (good!) and looks OK to me (from only looking at it, not diving into the code or trying it out). What do you mean with "fix to export to pdf"? This looks like something that should go into a commit message, not the source code.

Care to explain shortly to an "outsider" like me what exactly your patch does and why it is needed? How does exporting a PDF from Write work after applying it, in terms of user interactions (i.e. how does one use it)?

comment:2 Changed 15 years ago by alsroot

  • Milestone changed from Unspecified by Release Team to 0.88

added 0.88 milestone to not mess it w/ 0.86 patches

comment:3 Changed 14 years ago by tomeu

  • Owner changed from tomeu to uwog
  • Status changed from new to assigned

comment:4 Changed 14 years ago by walter

  • Milestone changed from 0.88 to 0.90

Alas, this doesn't seem to have landed. But whereas Fructose is not gated by Sucrose, this could land whenever.

comment:5 Changed 14 years ago by erikos

  • Keywords r! added; r? removed

Thanks Gonzalo for following up on this one.

I wonder a bit, why it was not enabled before, was the API not there in previous versions of abiword? Or we just forgot about it? Can you dig that up?

Btw, this patch does ruin your pep8 cleanups :)

comment:6 Changed 14 years ago by godiard

  • Owner changed from uwog to godiard

comment:7 Changed 13 years ago by RafaelOrtiz

Seeing sources this has landed on git, export to pdf is enabled.

related to:

comment:8 Changed 13 years ago by RafaelOrtiz

  • Resolution set to fixed
  • Status changed from assigned to closed

comment:9 Changed 11 years ago by dnarvaez

  • Milestone 0.90 deleted

Milestone 0.90 deleted

Note: See TracTickets for help on using tickets.