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

Disable borders in Word Export #42

Merged
merged 2 commits into from
May 7, 2024
Merged

Conversation

aror92
Copy link
Contributor

@aror92 aror92 commented May 3, 2024

  • Comment out handling of borders in Word Export

Borders do not currently display in FLEx.
But once a border is added to a style in FLEx, deselecting the border does not actually remove the border object from the FLEx style. This means if a border has ever been added in FLEx, it will still exist in the configuration node and be added in the word export. Until this is fixed, we should disable borders in the word export.


This change is Reviewable

- Comment out handling of borders in Word Export

Borders do not currently display in FLEx.
But once a border is added to a style in FLEx, deselecting
the border does not actually remove the border object from
the FLEx style. This means if a border has ever been added
in FLEx, it will still exist in the configuration node
and be added in the word export. Until this is fixed, we
should disable borders in the word export.

Change-Id: I19063eaaa9bb85f110ddb4c177958a7887e3af8c
@aror92 aror92 enabled auto-merge (squash) May 3, 2024 21:57
Copy link
Contributor

@mark-sil mark-sil left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @aror92)

@aror92 aror92 merged commit 3c13042 into release/9.1 May 7, 2024
5 checks passed
@aror92 aror92 deleted the bugfix/wordExportBorders branch May 7, 2024 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants