feat: [Table] enhancements

[Sumitmaithani]

Description

Fixes : #2319

Changes

added two new props

  /**
   * The contentAlign prop determines the content alignment of the table.
   * The contentAlign prop can be 'left', 'center', or 'right'.
   * The default value is `left`.
   **/
  contentAlign?: 'left' | 'center' | 'right';
  /**
   * The allowMultiline prop determines whether the table should allow multiline cells or not.
   * The default value is `false`.
   **/
  allowMultiline?: boolean;

Additional Information

Component Checklist

• Update Component Status Page
• Perform Manual Testing in Other Browsers
• Add KitchenSink Story
• Add Interaction Tests (if applicable)
• Add changeset

[changeset-bot]

⚠️ No Changeset found

Latest commit: 57a5f46

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[github-actions]

✅ PR title follows Conventional Commits specification.

[codesandbox-ci]

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 57a5f46:

Sandbox	Source
razorpay/blade: basic	Configuration

[Sumitmaithani]

Hi @saurabhdaware @snitin315 @chaitanyadeorukhkar can you please review this PR? It's been a while raising this PR. Thank you

[saurabhdaware]

Hi, sorry we missed reviewing this. Will get it reviewed and merged this week. Added one small comment meanwhile. Will check storybook locally in some time.

[saurabhdaware]

Maybe we can name this textAlign? since that is a more intuitive CSS prop for this?

[Sumitmaithani]

done 👍

[saurabhdaware]

@anuraghazra So the thing is with multiline cell, the editable inputs will break since inputs can only be of defined sizes. Lets discuss this with design once.

@Sumitmaithani will keep you posted. Sorry for the slow reviews as it needs some discussions

[saurabhdaware]

Hey @Sumitmaithani, we discussed this issue (#2322 (comment)) with our design team

So ideally we will have to change the height of inputs in TableEditableCell to match the full height similar to excel sheets.

Since this is a bit complex change based on how our BaseInput is structured. I think its better to keep this PR for textAlign only and skip the allowMultiline changes for now. We can pick it up later in the team or you can attempt it as separate PR if you're interested.

Let me know if this makes sense.

[saurabhdaware]

We'll also need textAlign on TableHeaderCell