Skip to content

Learn more helper dialog for background fit#854

Merged
wassgha merged 3 commits intomasterfrom
fill/learn-more
Mar 30, 2020
Merged

Learn more helper dialog for background fit#854
wassgha merged 3 commits intomasterfrom
fill/learn-more

Conversation

@wassgha
Copy link
Copy Markdown
Contributor

@wassgha wassgha commented Mar 30, 2020

Closes #251

Screen Shot 2020-03-30 at 1 56 37 AM

Changes

  • Implements a helper dialog for the background display panel
  • Internally refactors dialogs and notes/hints

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 30, 2020

Size Change: +793 B (0%)

Total Size: 503 kB

Filename Size Change
assets/js/edit-story.js 433 kB +793 B (0%)
ℹ️ View Unchanged
Filename Size Change
assets/css/edit-story.css 3.01 kB 0 B
assets/css/stories-dashboard.css 206 B 0 B
assets/js/stories-dashboard.js 67.2 kB 0 B

compressed-size-action

Copy link
Copy Markdown
Collaborator

@swissspidy swissspidy left a comment

Choose a reason for hiding this comment

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

Some nits, otherwise looks good.

@swissspidy
Copy link
Copy Markdown
Collaborator

Just realized it's the renaming of backgroundDisplay.js that breaks the test/backgroundDisplay.js test.

@wassgha
Copy link
Copy Markdown
Contributor Author

wassgha commented Mar 30, 2020

Fixed and rebased :))

@wassgha wassgha merged commit 3f1934e into master Mar 30, 2020
@wassgha wassgha deleted the fill/learn-more branch March 30, 2020 23:23
@swissspidy swissspidy added the Type: Enhancement New feature or improvement of an existing feature label Mar 31, 2020
obetomuniz added a commit that referenced this pull request Apr 1, 2020
* master: (43 commits)
  Bump @storybook/react from 5.3.17 to 5.3.18 (#902)
  Bump @storybook/addon-backgrounds from 5.3.17 to 5.3.18 (#903)
  updating import path for typeahead menu options
  Bump @storybook/addon-knobs from 5.3.17 to 5.3.18 (#899)
  Bump lint-staged from 10.1.0 to 10.1.1 (#898)
  Bump @storybook/addon-docs from 5.3.17 to 5.3.18 (#896)
  Bump @testing-library/dom from 7.1.4 to 7.2.0 (#890)
  Bump eslint-plugin-testing-library from 3.0.0 to 3.0.1 (#887)
  Fix drop targets on background element + disable opacity for backgrounds (#885)
  Enables snap to common element distances and removes workaround (#884)
  Shape previews in the layers panel (#853)
  [ImgBot] Optimize images
  Learn more helper dialog for background fit (#854)
  Restore defaultBackground migration
  fix output test for font-size
  minor lints
  New aspect ratio 2:3
  Fix: run the default-background migration (#880)
  Fixes background element selection (#847)
  Bump browserslist from 4.11.0 to 4.11.1
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Type: Enhancement New feature or improvement of an existing feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Learn More Dialogue

5 participants