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

AddTable setStyleName and custom style #1666

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

chrisribe
Copy link

Description

Added missing tblStyle tag when setting custom TableStyle.
This would limit tables to only accept a named style OR a custom one (array).
Adding this allows a user to select an existing table style defined in a template and set / override others (ex: layout, width, etc).

Fixes # (issue)
#629

Checklist:

  • [o ] I have run composer run-script check --timeout=0 and no errors were reported
  • [ x] The new code is covered by unit tests (check build/coverage for coverage report)
  • [ o] I have updated the documentation to describe the changes

chrisribe and others added 4 commits April 26, 2019 15:23
… into develop

Updated my fork with original project
Getting latest develop branch changes
Allows user to set both styleName and table style. Currently only one or the other.
Ex:
$section->addTable(array(
      "styleName" => "TBL2",
      "layout" => Table::LAYOUT_FIXED,
      "width"  => 75 * 50,
        "unit"   => TblWidth::PERCENT
));
@PowerKiKi PowerKiKi changed the base branch from develop to master November 16, 2022 21:11
@Progi1984 Progi1984 force-pushed the master branch 3 times, most recently from 2d9f999 to e458249 Compare August 30, 2023 11:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant