Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make all options work #10

Open
3 of 5 tasks
bertfrees opened this issue Apr 12, 2016 · 4 comments
Open
3 of 5 tasks

Make all options work #10

bertfrees opened this issue Apr 12, 2016 · 4 comments
Assignees
Milestone

Comments

@bertfrees
Copy link
Contributor

bertfrees commented Apr 12, 2016

  • All options defined in sbs:dtbook-to-pef should get a default value.
  • Options that are not wanted should be deleted from sbs:dtbook-to-pef.
  • Options that have been implemented in the generic dtbook-to-pef (see dtbook-to-pef.xpl and default.scss) should not be overwritten. For example, the default CSS should not contain the rule @page { size: 28 28 }. Instead provide default values for the options page-width and page-height.
  • Options that were not generic enough to implement in the generic dtbook-to-pef should be implemented here. (This includes options that must be implemented in every agency script, such as show-braille-page-numbers and show-print-page-numbers, but also the options that are probably only useful for SBS and should therefore best be removed from the generic script: accented-letters, polite-forms, downshift-ordinal-numbers, etc..)
  • Ideally all options, especially the ones implemented here, should be covered by unit tests.
@egli
Copy link
Member

egli commented Sep 1, 2016

I cannot find any problems with the third issue about not overwriting generic options. In particular there is nothing in the default.scss about page size. A textual inspection doesn't reveal any other problems. So checking that problem off.

@bertfrees
Copy link
Contributor Author

There is at least one issue:

hyphens: auto; /* FIXME: don't hard-code, depends on param

egli added a commit that referenced this issue Sep 2, 2016
bertfrees added a commit that referenced this issue Oct 14, 2016
bertfrees added a commit to sbsdev/pipeline that referenced this issue Apr 5, 2017
bertfrees added a commit to sbsdev/pipeline that referenced this issue May 4, 2017
bertfrees added a commit to sbsdev/pipeline that referenced this issue May 5, 2017
@bertfrees
Copy link
Contributor Author

I'm going to take over this issue from Christian.

Below is the complete list of options. I'll create a small test for each of them.

  1. Implemented in the SBS specific dtbook-to-pef (or partly in generic script, partly in SBS script):

    • contraction-grade
    • document-identifier
    • footnotes-placement
    • include-production-notes
    • levels-in-footer
    • show-braille-page-numbers
    • show-print-page-numbers
  2. To be implemented in the SBS specific dtbook-to-pef:

    • accented-letters
    • colophon-metadata-placement
    • downshift-ordinal-numbers
    • include-image-groups
    • include-note-references
    • left-margin
    • main-document-language
    • minimum-number-of-sheets
    • number-of-sheets
    • polite-forms
    • rear-cover-placement
    • tab-width
    • text-level-formatting
  3. Implemented in the generic dtbook-to-pef:

    • ascii-file-format
    • capital-letters
      • also tested in mod-sbs
    • duplex
      • also tested in mod-sbs
    • force-braille-page-break
      • also tested in mod-sbs
    • hyphenation
      • also tested in mod-sbs
    • include-brf
    • include-captions
      • also tested in mod-sbs
    • include-images
      • also tested in mod-sbs
    • include-line-groups
      • also tested in mod-sbs
    • include-preview
    • line-spacing
      • also tested in mod-sbs
    • maximum-number-of-sheets
      • also tested in mod-sbs
    • page-height
      • also tested in mod-sbs
    • page-width
      • also tested in mod-sbs
    • stylesheet
    • toc-depth
      • also tested in mod-sbs

@bertfrees bertfrees assigned bertfrees and unassigned egli Sep 4, 2017
@bertfrees
Copy link
Contributor Author

@egli You have checked the box above that says all options that are not wanted have been removed from sbs:dtbook-to-pef." Just want to double check that is true for these options:

  • ascii-file-format
  • left-margin
  • capital-letters
  • include-captions
  • include-image-groups
  • include-images
  • include-line-groups
  • include-note-references
  • include-production-notes
  • main-document-language
  • rear-cover-placement
  • tab-width
  • text-level-formatting
  • etc.

@mixa72

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants