Use 12-column layout for dashboard grid

[eradman]

What type of PR is this?

• Feature

Description

• Set minSizeX, minSizeY for widgets to 2 since a value of 1 breaks all assumptions of the UI layout.
• Migration provide transition from 6 to 12 columns for all widgets.

How is this tested?

• E2E Tests (Cypress)
• Manually

./manage.py db downgrade 1655999df5e3
./manage.py db upgrade

Related Tickets & Documents

#4809

Mobile & Desktop Screenshots/Recordings (if there are UI changes)

12-column.mov

[eradman]

@yoshiokatsuneo if you have time, please review this change. This is a feature that users I support have asked for, and I think others would fit it useful as well.

[yoshiokatsuneo]

@yoshiokatsuneo if you have time, please review this change. This is a feature that users I support have asked for, and I think others would fit it useful as well.

@eradman

Thank you for the cool PR ! The code looks good and the change is straightforward.
I tested your code and confirmed that the dashboard is working with 12 columns without problem.
Also, I tested and confirmed that the migration works well !

One thing I notice is that the default widget width is narrower than before, as it is half of what is was before like below.
Is it intended behavior ?
If it is not intended, I think we can double defaultSizeX, and defaultColumns for each visualizations.

Thank you !

[eradman]

If it is not intended, I think we can double defaultSizeX, and defaultColumns for each visualizations.

Good observation. I updated defaultSizeX as well

[eradman]

Found out that most visualizations have their own default width. So I need to update these as well

viz-lib/src/visualizations/chart/index.ts:  defaultColumns: 3,
viz-lib/src/visualizations/choropleth/index.ts:  defaultColumns: 3,
viz-lib/src/visualizations/counter/index.ts:  defaultColumns: 2,
viz-lib/src/visualizations/details/index.ts:  defaultColumns: 2,
viz-lib/src/visualizations/map/index.ts:  defaultColumns: 3,
viz-lib/src/visualizations/pivot/index.ts:  defaultColumns: 3,
viz-lib/src/visualizations/table/index.ts:  defaultColumns: 3,

[eradman]

Ready for review

[yoshiokatsuneo]

Thank you very much !
I think this PR makes Redash more useful !

I just approved the PR so that you can merge after updating branch.

[eradman]

Thank you @yoshiokatsuneo! Setting to auto-merge

[yoshiokatsuneo]

Just as a reference.
There is a user who does not have col option in the widgets table got error.
I'm not sure why there is no col option.

#7494 (comment)

[eradman]

There is a user who does not have col option in the widgets table got error.

We'll keep an eye on this discussion: I don't see how it's possible to add a widget to a dashboard without setting the "col" value.