Skip to content

Conversation

@Razi91
Copy link

@Razi91 Razi91 commented Mar 31, 2022

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Documentation
  • Code style update
  • Refactor
  • Build-related changes
  • Other, please describe:

Does this PR introduce a breaking change?

  • Yes
  • No

The PR fulfills these requirements:

  • It's submitted to the dev branch (or v[X] branch)
  • When resolving a specific issue, it's referenced in the PR's title (e.g. fix: #xxx[,#xxx], where "xxx" is the issue number)
  • It's been tested on a Cordova (iOS, Android) app
  • It's been tested on a Electron app
  • Any necessary documentation has been added or updated in the docs or explained in the PR's description.

If adding a new feature, the PR's description includes:

  • A convincing reason for adding this feature (to avoid wasting your time, it's best to start a new feature discussion first and wait for approval before working on it)

Other information:

Simplifies using expand on QTable by defining just the additional row, instead of rewriting the whole body slot, described in #8350 by adding slot row-expand with tr inside. This tr will be added always, in this place user should decide if it should by hidden by v-show or v-if.

@Razi91 Razi91 changed the title QTable slot row-expand #8350 feat(ui/QTable) QTable slot row-expand #8350 Mar 31, 2022
@Razi91 Razi91 force-pushed the QTable-slot-expanded branch 3 times, most recently from 6851f10 to a46a5b0 Compare August 24, 2023 22:05
@Razi91
Copy link
Author

Razi91 commented Aug 24, 2023

Updated and fixed for virtualscroll. However, I'm not sure about performance, especially with virtualscroll. It shouldn't provide any breaking change, the slot is completely opt-in.

{
name: 'before',
format: () => ''
},

Choose a reason for hiding this comment

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

In QTableProps['columns'] type, name, label and field are required

const columns = [
{
name: 'before',
format: () => ''

Choose a reason for hiding this comment

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

In QTableProps['columns'] type, name, label and field are required

const columns = [
{
name: 'before',
format: () => ''

Choose a reason for hiding this comment

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

In QTableProps['columns'] type, name, label and field are required

},

"row-expand": {
"desc": "Slot to define how a body row looks like; Suggestion: QTr + Td with `colspan=\"100%\"`",

Choose a reason for hiding this comment

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

Slot to define how the expanded part of a row looks like; Suggestion: QTr + Td with colspan=\"100%\"

@Razi91 Razi91 force-pushed the QTable-slot-expanded branch from 3537f04 to 352b12f Compare November 13, 2023 20:51
@Theophile-BOUCHER
Copy link

Hello everyone,
For my part, the row-expand slot is eagerly awaited!
So when do you plan to merge this pull request? Overloading the body and header would be wasted work if this PR was ever merged

@Razi91
Copy link
Author

Razi91 commented Feb 12, 2024

Still waiting, fixed the comments in feedback, now still waiting....

@maximang
Copy link

I'm not part of the Quasar team. I just made few comments in order to help the PR getting approved.

@Miosss
Copy link

Miosss commented Mar 20, 2024

+1, expand-row slot would be a great addition.
Now you would have to overwrite entire row slot which is not great. body-cell-[] slots are great for styling certain columns only, but they do not work well with current approach to expanding rows.

@RichAyotte
Copy link

What's needed to get this merged besides resolving the branch conflict?

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.

9 participants