Skip to content
This repository has been archived by the owner on May 26, 2022. It is now read-only.

Add setCellVerticalAlignment() & allow setShouldWrapText() to be explicitly set as false #869

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Commits on Feb 12, 2022

  1. Add setCellVerticalAlignment() & allow setShouldWrapText() to be expl…

    …icitly set as false
    
    - add Common\Entity\Style\CellVerticalAlignment
    - duplicate get/set/hasSet/shouldApply methods for CellAlignment in Common\Entity\Style\Style for CellVerticalAlignment instead, plus corresponding properties
    - add setCellVerticalAlignment method to Common\Creator\Style\StyleBuilder
    - add vertical alignment to StyleMerger:: mergeCellProperties()
    - adjust wrapText logic in mergeCellProperties() to fix issue box#829
    - apply vertical cell styling for both XLSX and ODS, via corresponding StyleManager classes
    - transform vertical alignment  ‘center’ to ‘middle’ for ODS
    - fix logic around wrapText such that the choice whether to include wrapping styles depends on hasSetWrapText() being true, and then use shouldWrapText() thereafter to either set wrapping or no wrapping (for XLSX, wrapText=“1” or wrapText=“0”, for ODS, wrap-option=wrap or wrap-option=no-wrap). previously there was no way to set wrapping to be OFF, only to set it to be ON.
    - add new tests to ensure shouldWrapText(false) results in the correct negated wrapText (XLSX) / wrap-option (ODS) styles
    - add new tests to StyleBuilderTest for vertical alignment
    - add vertical alignment to documentation.md
    jonnott committed Feb 12, 2022
    Configuration menu
    Copy the full SHA
    887d6ef View commit details
    Browse the repository at this point in the history