cute-jumper / org-table-sticky-header

Sticky header for org-mode tables
36 stars 4 forks source link

Does not respect column width and alignment #11

Closed thecountoftuscany closed 5 years ago

thecountoftuscany commented 5 years ago

The sticky header does not respect column widths (defined with \<number>) as mentioned here so the header and table contents are not aligned properly and look skewed.

image

stardiviner commented 5 years ago

I have similar issue:

image

stardiviner commented 5 years ago

I fixed it by small modification.

Here is the screenshot:

image

I will create a PR soon.

stardiviner commented 5 years ago

This is caused by customize org-table face.

thecountoftuscany commented 5 years ago

@cute-jumper Is there some issue with this PR?

cute-jumper commented 5 years ago

Yep. I had a minor question for the PR. Not getting response from @stardiviner (BTW, in general, I think the PR looks great)

stardiviner commented 5 years ago

@cute-jumper Aha, sorry for missing this PR comment. The :underline t is not required. Just help user to see by separate the header line from table lines.

stardiviner commented 5 years ago

@cute-jumper The latest version missed two parentheses at the end of function. My fault. I pushed a fix commit on my repo. will add PR soon. UPDATE: https://github.com/cute-jumper/org-table-sticky-header/pull/13