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

Align content vertically in restricted widget type #4056

Merged
merged 1 commit into from
Aug 11, 2019

Conversation

osule
Copy link
Contributor

@osule osule commented Aug 11, 2019

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • New Query Runner (Data Source)
  • New Alert Destination
  • Other

Description

This displays content in restricted widget type vertically centered.
It sets the flex property for the scrollbox back to default initial value
by removing body-row-auto class.

Related Tickets & Documents

#3949

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

Mobile
image

Desktop
image

This displays content in restricted widget type vertically centered.
It sets the flex property for the scrollbox back to default initial value
by removing `body-row-auto` class.

Fixes getredash#3949
@arikfr arikfr requested a review from ranbena August 11, 2019 11:01
@ranbena ranbena merged commit 4e5f55a into getredash:master Aug 11, 2019
@ranbena ranbena mentioned this pull request Aug 14, 2019
1 task
gabrieldutra added a commit that referenced this pull request Aug 18, 2019
arikfr pushed a commit that referenced this pull request Sep 20, 2019
* Improve sizing for Number inputs

Co-Authored-By: Ran Byron <[email protected]>

* Migrate WidgetDialog

* Start migrating Widget

* Update textbox to use HtmlContent

* QueryLink migration and some updates

* Add visualization rendering

* Render widget

* Add delete button

* Update AutoHeight

* Add widget bottom

* Add Drodpown button

* Split Widget component

* Update with #4056 and trigger netlify

* In progress: use composition

* Add header and footer

* Update widget actions positioning

* Re-render when refreshing from widget

* Add workaround to force DashboardGrid re-render

* VisualizationWidgetFooter component

* VisualizationWidget menu

* Separate RestrictedWidget

* Update tests

* Update margin for Parameters

* Remove widget files

* Revert "Improve sizing for Number inputs"

This reverts commit a02ce8f.

* Some cleanup

* Move refresh logic to the Dashboard

* Add loadingWidgets logic to the public dashboard

* Add onLoadWidget

* Remove parameter from URL when empty

* Recreate widget array instead of loadingWidgets

* Add comment about re-rendering + whitespace missing

* CR changes

* Use plain html instead of string syntax

Co-Authored-By: Ran Byron <[email protected]>
harveyrendell pushed a commit to pushpay/redash that referenced this pull request Nov 14, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants