Skip to content

Base Styles: Import colors into variables#19159

Merged
ockham merged 4 commits intoWordPress:masterfrom
ockham:add/base-styles-variables-colors-import
Dec 16, 2019
Merged

Base Styles: Import colors into variables#19159
ockham merged 4 commits intoWordPress:masterfrom
ockham:add/base-styles-variables-colors-import

Conversation

@ockham
Copy link
Copy Markdown
Contributor

@ockham ockham commented Dec 16, 2019 •
edited
Loading

Description

@import "./colors"; from within _variables.scss, since the latter depends on $dark-gray-900 which is defined in the former.

Found while working on Automattic/wp-calypso#38445.

How has this been tested?

Try importing just @wordpress/base-styles/variables in your app, using the currently released package. You'll get a build error saying something like

    Module build failed (from ./node_modules/sass-loader/dist/cjs.js):
    SassError: Undefined variable: "$dark-gray-900".

Now try with the changes that this PR makes. Your style files should build successfully 👍

Types of changes

Bug fix.

Checklist:

  • My code is tested.
  • My code follows the WordPress code style.
  • My code follows the accessibility standards.
  • My code has proper inline documentation.
  • I've included developer documentation if appropriate.
  • I've updated all React Native files affected by any refactorings/renamings in this PR. .

@ockham ockham requested a review from simison December 16, 2019 10:52
@ockham ockham self-assigned this Dec 16, 2019
@ockham ockham requested a review from sirreal December 16, 2019 10:52
@simison
Copy link
Copy Markdown
Member

simison commented Dec 16, 2019 •
edited
Loading

Hmm right, I suppose elsewhere we've always been importing all the files at once so that all variables always exist. I.e. as highlighted in docs:

https://github.com/WordPress/gutenberg/tree/bfd1117846c7802d62dbba046c27968d961781e7/packages/base-styles#scss-utilities-and-variables

Things obviously won't work when importing just one file directly like in your PR.

The build/test step during Gutenberg-CI runs won't catch these issues either — should it?

@ockham ockham mentioned this pull request Dec 16, 2019
Closed
@simison simison requested a review from gziolo December 16, 2019 11:05
@ockham
Copy link
Copy Markdown
Contributor Author

ockham commented Dec 16, 2019

CircleCI's Lint job is failing, complaining that it

 Expected double quotes

Which I find a bit surprising? Is Gutenberg using double quotes in SASS imports?

@ockham
Copy link
Copy Markdown
Contributor Author

ockham commented Dec 16, 2019

Hmm right, I suppose elsewhere we've always been importing all the files at once so that all variables always exist. I.e. as highlighted in docs:

https://github.com/WordPress/gutenberg/tree/bfd1117846c7802d62dbba046c27968d961781e7/packages/base-styles#scss-utilities-and-variables

Things obviously won't work when importing just one file directly like in your PR.

The build/test step during CI runs won't catch these either — should it?

I think that'd make sense 👍

@gziolo gziolo added the [Type] Bug An existing feature does not function as intended label Dec 16, 2019
@simison
Copy link
Copy Markdown
Member

simison commented Dec 16, 2019 •
edited
Loading

lint error [...] Which I find a bit surprising? Is Gutenberg using double quotes in SASS imports?

Yeah — Gutenberg prolly uses https://www.npmjs.com/package/stylelint-config-wordpress which is double quotes indeed, if I remember correctly.

@ockham
Copy link
Copy Markdown
Contributor Author

ockham commented Dec 16, 2019

lint error [...] Which I find a bit surprising? Is Gutenberg using double quotes in SASS imports?

Yeah — Gutenberg prolly https://www.npmjs.com/package/stylelint-config-wordpress which is double quotes indeed, if I remember correct.

Pushed a fix 👍

@ockham ockham changed the title Add/base styles variables colors import Base Styles: Import colors into variables Dec 16, 2019
@simison
Copy link
Copy Markdown
Member

simison commented Dec 16, 2019 •
edited
Loading

The build/test step during CI runs won't catch these either — should it?

I think that'd make sense 👍

I personally think we should render each file separately as a test run and ensure;

More convo about testing in original PR #17883 — see commits for details and an implementation example. I don't think that needs to be done in this PR tho but prolly easy to catch (and fix) any other missing imports that way.

gziolo
gziolo reviewed Dec 16, 2019
View reviewed changes
@ockham @gziolo
Changelog: Clarify that it's a Bug Fix
90a0f40 
Co-Authored-By: Grzegorz (Greg) Ziółkowski <grzegorz@gziolo.pl>
gziolo
gziolo approved these changes Dec 16, 2019
View reviewed changes
Copy link
Copy Markdown
Member

@gziolo gziolo left a comment

Choose a reason for hiding this comment

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

It would be nice to include a section title in the changelog to make it easier to read. Thanks for the fix 💯

@ockham ockham merged commit f4ba3d1 into WordPress:master Dec 16, 2019
@ockham ockham deleted the add/base-styles-variables-colors-import branch December 16, 2019 11:55
@hypest
Copy link
Copy Markdown
Contributor

hypest commented Dec 16, 2019

It seems that the native mobile side is broken by this change, with this error:

Error: ./colors could not be resolved in /home/circleci/project/gutenberg/packages/edit-post/src/components/layout,/home/circleci/project,gutenberg/packages/edit-post/src/components/layout,/home/circleci/project/src,/home/circleci/project/gutenberg/assets/stylesheets,/home/circleci/project/gutenberg/packages/base-styles

My first impression is that the SASS transformer we use in the RN app pipeline doesn't understand partial SASS files, and is unable to resolve the (existing) ./_colors.scss.

@simison
Copy link
Copy Markdown
Member

simison commented Dec 16, 2019 •
edited
Loading

@hypest is the solution to import ./_colors.scss (i.e. include the file extension) in Gutenberg or change something at your side?

@hypest
Copy link
Copy Markdown
Contributor

hypest commented Dec 16, 2019

👋 @simison!

On the Gutenberg-web side, including the underscore is fixing it for native. The native mobile SASS transformer apparently doesn't support partial SASS files so, it doesn't automatically try to resolve the underscore-less file to the underscored one.

If handled on the native mobile side alone, a solution seems to be to introduce a new file colors.native.scss, and in it just @import "./_colors". It's a bit hacky but, can be a workaround.

I wonder though, perhaps it's an obsolete impression but, I thought that the colors.scss files were automatically imported for all SASS files so, one didn't need to explicitly import them 🤔. That's what we had implemented in native mobile too, automatically import _colors.scss and others.

@ockham
Copy link
Copy Markdown
Contributor Author

ockham commented Dec 16, 2019

I wonder though, perhaps it's an obsolete impression but, I thought that the colors.scss files were automatically imported for all SASS files so, one didn't need to explicitly import them . That's what we had implemented in native mobile too, automatically import _colors.scss and others.

The point of this PR was to enable 3rd party projects to import individual files from @wordpress/base-styles (specifically, _variables.scss) by making the -- otherwise implicit -- styling dependency chain explicit. Without the colors import, just importing _variables.scss wouldn't work, since the 3rd party project wouldn't know where to find the definition for $dark-gray-900, which is used in _variables.scss.

@ockham
Copy link
Copy Markdown
Contributor Author

ockham commented Dec 16, 2019

@hypest Are you sure about the partials support thing? I thought that the underscore/non-underscore aliasing was built into SASS at a pretty low level (but I might be wrong).

@ockham
Copy link
Copy Markdown
Contributor Author

ockham commented Dec 16, 2019

It seems that the native mobile side is broken by this change, with this error:

Error: ./colors could not be resolved in /home/circleci/project/gutenberg/packages/edit-post/src/components/layout,/home/circleci/project,gutenberg/packages/edit-post/src/components/layout,/home/circleci/project/src,/home/circleci/project/gutenberg/assets/stylesheets,/home/circleci/project/gutenberg/packages/base-styles

My first impression is that the SASS transformer we use in the RN app pipeline doesn't understand partial SASS files, and is unable to resolve the (existing) ./_colors.scss.

@hypest Do you have some more context on this (e.g. failing CI job link)? 🙂
It's kinda weird that it's not resolving the file correctly...

@ockham
Copy link
Copy Markdown
Contributor Author

ockham commented Dec 16, 2019

Ah I think I found the relevant file https://github.com/wordpress-mobile/gutenberg-mobile/blob/06b924d832e1bdd5bcbb2961f8c5e2051d772269/sass-transformer.js

@hypest
Copy link
Copy Markdown
Contributor

hypest commented Dec 16, 2019

@hypest Are you sure about the partials support thing? I thought that the underscore/non-underscore aliasing was built into SASS at a pretty low level (but I might be wrong).

I'm not too sure, I've only assuming that the SASS transformer we use in the RN mobile port (this library: https://github.com/kristerkari/react-native-sass-transformer) doesn't support it because if I just add the underscore the import is resolved OK.

@hypest Do you have some more context on this (e.g. failing CI job link)? 🙂
It's kinda weird that it's not resolving the file correctly...

The only CI indication that this fails is on the gutenberg-mobile repo for now. See this https://app.circleci.com/jobs/github/wordpress-mobile/gutenberg-mobile/20081 and search for "colors could not be resolved".

@ockham
Copy link
Copy Markdown
Contributor Author

ockham commented Dec 16, 2019

Looks like it's coming from gutenberg-mobile itself: https://github.com/wordpress-mobile/gutenberg-mobile/blob/06b924d832e1bdd5bcbb2961f8c5e2051d772269/sass-transformer.js#L139

Can we maybe just make that thing smarter? https://github.com/wordpress-mobile/gutenberg-mobile/blob/06b924d832e1bdd5bcbb2961f8c5e2051d772269/sass-transformer.js#L133

@ockham
Copy link
Copy Markdown
Contributor Author

ockham commented Dec 16, 2019

Looks like it's coming from gutenberg-mobile itself: https://github.com/wordpress-mobile/gutenberg-mobile/blob/06b924d832e1bdd5bcbb2961f8c5e2051d772269/sass-transformer.js#L139

(I guess that's our local fork of https://github.com/kristerkari/react-native-sass-transformer)

@hypest
Copy link
Copy Markdown
Contributor

hypest commented Dec 16, 2019

Can we maybe just make that thing smarter? https://github.com/wordpress-mobile/gutenberg-mobile/blob/06b924d832e1bdd5bcbb2961f8c5e2051d772269/sass-transformer.js#L133

Perhaps. It feels that'd be equivalent to trying to add support for partials 😬. Maybe we can live with just trying out the underscored filename.

(I guess that's our local fork of https://github.com/kristerkari/react-native-sass-transformer)

Indeed.

@ockham
Copy link
Copy Markdown
Contributor Author

ockham commented Dec 16, 2019

Can we maybe just make that thing smarter? https://github.com/wordpress-mobile/gutenberg-mobile/blob/06b924d832e1bdd5bcbb2961f8c5e2051d772269/sass-transformer.js#L133

Perhaps. It feels that'd be equivalent to trying to add support for partials 😬

Reading https://sass-lang.com/guide, I got the impression that there isn't that much to partials, other than a nomenclature to signal that they shouldn't be turned into CSS on their own (but used as imports elsewhere), and the leading underscore alias resolution 🤷‍♂️

Maybe we can live with just trying out the underscored filename.

Yeah, that might work as well.

(I guess that's our local fork of https://github.com/kristerkari/react-native-sass-transformer)

Indeed.

👍

@ockham
Copy link
Copy Markdown
Contributor Author

ockham commented Dec 16, 2019

Turns out upstream already has support for partials: https://github.com/kristerkari/react-native-sass-transformer/blob/52884dd59582856fa17e2b2e8ca9efc37d412387/index.js#L41

ockham added a commit to ockham/gutenberg-mobile that referenced this pull request Dec 16, 2019
@ockham
Sass Transformer: Also look for partials
0002811 
Add support for [SASS partials](https://sass-lang.com/guide), as is already present [upstream](https://github.com/kristerkari/react-native-sass-transformer/blob/52884dd59582856fa17e2b2e8ca9efc37d412387/index.js#L41-L42).

This should fix the issue introduced by WordPress/gutenberg#19159, and discussed there.
@ockham ockham mentioned this pull request Dec 16, 2019
Merged
2 tasks
@youknowriad youknowriad added this to the Gutenberg 7.2 milestone Jan 6, 2020
Tug added a commit that referenced this pull request Jan 6, 2020
@dratwas @Tug
@mchowning @etoledom @SergioEstevao @cameronvoell @koke @JavonDavis @maxme @mkevins @hypest @marecar3 @lukewalczak @pinarol @designsimply @jbinda @ockham
[Monorepo][base-branch] import of gutenberg-mobile (#18508)
26ee90d 
* Use string-array instead of plurals tag in strings.xml

See https://github.com/GlotPress/GlotPress-WP/blob/master/gp-includes/formats/format-android.php

* Update string files

* Update scripts to minimize changes in git diff and keep the same context for android

* Exclude strings from tests

* Fix lint errors

* Bump version to 1.17.1

* Update gutenberg ref following 1.17 merge to gutenberg master

* Update gutenberg ref

* Remove declaration on bridge of unused methods.

* Added bridge code for gutenberg to request a native fullscreen preview for for an image from a URL on iOS.

* Updating bundle and gutenberg reference.

* Updated release notes.

* Update gutenberg ref

* Update gutenberg ref

* Update gutenberg ref

* Update GB reference.

* Updating bundle after catching up branch and gutenberg submodule.

* Update gutenberg ref

* Update release notes.

* Update UI test

* Pass postType as initial prop on iOS

* Pass postType down to Editor

* Allow Android to set the post type

* Updating gutenberg reference.

* Removed duplicate line from merge.

* updafe test files and iOS version for running locally with xcode 11

* Update gutenberg ref

* Update gutenberg ref

* Update aztec to version that support reversed and start on lists.

* Update Gutenberg to version where list settings are active in native.

* update block list check and capabilities

* replace double click on android

* fix block insertion timeout

* update branch with develop

* Update gutenberg ref

* Update package.json and JS bundle for 1.18.0 release

* Update Gutenberg

* Update Gutenberg

* update gutenberg ref

* Update GB-reference.

* Update release notes.

* revert caps to iPhone Simulator

* Update Gutenberg ref

* Update Release notes

* Update gutenberg/ reference

* Update bundles

* Update GB reference.

* Fix spacing

* Add static method to Media class to create instance using mimeType

* Add flag to track when appending multiple selected media items as blocks

* Introduce mediaSelectionCancelled method in WPAndroidGlueCode

* Set flag to append blocks when multiple = false is not respected

* Only use appendUploadMediaFiles plural version

* Update gutenberg ref

* Update to latest Gutenberg master

* Patch jsdom to implement Element.closest()

* Bring back changes on package.json from 1.7.1

* Add docstring to the function

* Return null as per https://dom.spec.whatwg.org/#dom-element-closest

* Update Aztec version.

* Update GB reference.

* Update gutenberg ref

* Update RELEASE-NOTES.txt

* Update GB reference

* Improve code block style

* Update GB reference.

* Update gutenberg ref

* Update Media mimeType truncation to use enum names

* Add Javadoc for mAppendsMultipleSelectedToSiblingBlocks flag

* Set appends to sibling blocks flag explicitly for all requests

* Add clarifying comment for special block append handling

* Remove singular (unused) appendMedia method

* Update GB reference.

* Update GB reference to master.

* Set appends to sibling blocks flag explicitly for other media pick

* Update GB reference.

* Update GB reference.

* Use lowercase for Media mimeType truncation

* Update Gutenberg

* Update Gutenberg

* Update Gutenberg ref - after fix for caption alignment

* Update bundles

* Point to aztecVersion hash which supports list with start and reverse attribute

* Update bundle and gutenberg ref - fix disappearing image

* Update GB reference.

* Update GB reference.

* Update GB reference.

* Update Gutenberg

* Update Gutenberg ref

* Update GB reference

* Update GB reference.

* Only enable page templates on dev builds

* Update aztec version

* Update aztec version

* Add colors for gallery block

* Update gutenberg reference

* Update gutenberg reference

* Update gutenberg reference

* Update GB reference.

* Update Gutenberg ref

* Update Aztec version to 1.14.0

* Make sure if the textColor is changed the default text color is updated.

* Use Slider from react-native-community lib (#1620)

* Use Slider from react-native-community lib

* Update slider version

* Add function to read npm version

* Rengenerate yarn.lock

* Bump node version

* Update readNPMVersion function

* Use Slider from fork

* Update Slider commit

* Use Slider from wordpress-mobile fork

* Add react-native-slider podspec

* Improve babel config

* Correct settings.gradle

* Correct project.pbxproj

* Update ref

* Bump

* Update ref

* Update ref to gutenberg master

* Update ref to gutenberg quick fix

* Update ref to gutenberg master

* Update Aztec editor version.

* Update to iOS 13

* Update Xcode version.

* Use iOS 12 for tests.

* Fix typo

* Update Aztec to fix CI error with xcode 10

* Use iOS 12.2

* Update Appium version.

* Update to appium 1.15.1

* Update to Appium 1.15.1 only in iOS

* Update caps.

* Add Gallery Block (#1498)

* WIP - initial-html.js for gallery testing

* Add parent app media mock for Android

* Update gutenberg reference

* Update gutenberg reference

* Comment out line setting mPendingMediaUploadCallback to null

* Update gutenberg reference

* Generate bundles

* Update gutenberg ref

* Generate bundles

* Generate bundles

* Add some color-studio colors for gallery

* Update gutenberg reference

* Generate bundle

* WIP update ref

* Update gutenberg reference

* Update gutenberg submodule

* Generate bundles

* Add $gray-40 color

* Update gutenberg reference

* Generate bundles

* Update gutenberg reference

* Generate bundles

* Update gutenberg reference

* Generate bundles

* Update gutenberg reference

* Generate bundles

* Update gutenberg reference

* Update gutenberg reference

* Update gutenberg reference

* Update gutenberg reference

* Update gutenberg reference

* Update gutenberg reference

* Update gutenberg reference

* Update gutenberg reference

* Restore demo content

* Restore anonymous implementation of GutenbergBridgeJS2Parent

* Update gutenberg reference

* Generate bundles

* Bump up Aztec version on iOS Example app

* Update gb ref

* Generate bundles

* Update gutenberg ref

* Update gutenberg ref (#1646)

* Add release notes for Gallery (#1658)

* Make sure we use iPhone 11 (iOS 13) for build and run tests

* Pooint aztecVersion to develop

* Update Appium for Android tests too.

* Set Appium to 1.15.0

* Update aztec version to 1.3.36

* Update appium to 1.16.0-rc.1

* Update Aztec iOS to 1.14.1

* Update GB reference.

* Activate preformat block on android platform (#1615)

* Updates package.json and JS bundle for 1.19.0 release.

* Update to shorten git commands

Make the git commands a little easier to copy by taking out `$` from the start of the lines.

This also matches with the other commands on the page which do not start with `$`.

* Update Unit Tests headings in Getting Started documentation

The heading "Test" should be "Unit Tests".

The heading "Writing and Running Tests" should be "Writing and Running Unit Tests".

https://github.com/wordpress-mobile/gutenberg-mobile/blob/develop/src/index.test.js

* Update gutenberg reference

* Update gutenberg reference

* Update gutenberg ref

* Updating bundles.

* Feat: Navigation Down in InnerBlocks (#1379)

* Add ref to gutenberg repo

* Fix e2e tests

* Update ref to gutenberg master

* Update Gutenberg ref

* Remove empty line between checkboxes

* Updating release notes to show video settings in 1.19

* Updating gutenberg reference to latest 1.19 release change on gutenberg master branch

* Update gutenberg ref

* Sass Transformer: Also look for partials

Add support for [SASS partials](https://sass-lang.com/guide), as is already present [upstream](https://github.com/kristerkari/react-native-sass-transformer/blob/52884dd59582856fa17e2b2e8ca9efc37d412387/index.js#L41-L42).

This should fix the issue introduced by #19159, and discussed there.

* Single quotes

* Also update sass-transformer-inside-gb.js

* Update release notes

* Update gutenberg ref

* Update gutenberg ref

* Update gutenberg ref

* Update gutenberg ref

* Update Gutenberg ref

* Update gutenberg ref

* Update gutenberg ref

* Bundles up to date with merged code from develop

* Add ref to gutenberg repo

* Update ref

* Update ref

* Update ref

* Upgrade the SVG lib to fix #1703

* Upgrade the Video lib to fix #1705

* Upgrade the Slider lib to use node_modules in local npm build

* Update ref

* Upgrade the SVG lib ref

* Upgrade the Video lib ref

* Upgrade the Slider lib ref

* Update ref

* Brings back gb master to normal (#1722)

* Fix/Bring back master to normal (#1724)

* Update Gutenberg ref

* Update Gutenberg ref

Co-authored-by: Tugdual de Kerviler <dekervit@gmail.com>
Co-authored-by: Matt Chowning <mchowning@gmail.com>
Co-authored-by: etoledom <etoledom@icloud.com>
Co-authored-by: Sérgio Estêvão <sergioestevao@gmail.com>
Co-authored-by: Cameron Voell <cameronvoell@gmail.com>
Co-authored-by: Jorge Bernal <jbernal@gmail.com>
Co-authored-by: Javon Davis <JavonDavis@users.noreply.github.com>
Co-authored-by: Maxime Biais <maxime@bia.is>
Co-authored-by: Gerardo Pacheco <gerardo.pacheco@automattic.com>
Co-authored-by: Matthew Kevins <mkevins@users.noreply.github.com>
Co-authored-by: Stefanos Togoulidis <stefanostogoulidis@gmail.com>
Co-authored-by: Marko Savic <savicmarko1985@gmail.com>
Co-authored-by: Luke Walczak <lukasz.walczak.pwr@gmail.com>
Co-authored-by: Pinar Olguc <pinarolguc@gmail.com>
Co-authored-by: Sheri Bigelow <sheri@designsimply.com>
Co-authored-by: jbinda <jakub.binda@gmail.com>
Co-authored-by: Bernie Reiter <ockham@raz.or.at>
Tug added a commit that referenced this pull request Jan 16, 2020
@dratwas @Tug
@mchowning @etoledom @SergioEstevao @cameronvoell @koke @JavonDavis @maxme @mkevins @hypest @marecar3 @lukewalczak @pinarol @designsimply @jbinda @ockham
[Monorepo][base-branch] import of gutenberg-mobile (#18508)
4cbda37 
* Use string-array instead of plurals tag in strings.xml

See https://github.com/GlotPress/GlotPress-WP/blob/master/gp-includes/formats/format-android.php

* Update string files

* Update scripts to minimize changes in git diff and keep the same context for android

* Exclude strings from tests

* Fix lint errors

* Bump version to 1.17.1

* Update gutenberg ref following 1.17 merge to gutenberg master

* Update gutenberg ref

* Remove declaration on bridge of unused methods.

* Added bridge code for gutenberg to request a native fullscreen preview for for an image from a URL on iOS.

* Updating bundle and gutenberg reference.

* Updated release notes.

* Update gutenberg ref

* Update gutenberg ref

* Update gutenberg ref

* Update GB reference.

* Updating bundle after catching up branch and gutenberg submodule.

* Update gutenberg ref

* Update release notes.

* Update UI test

* Pass postType as initial prop on iOS

* Pass postType down to Editor

* Allow Android to set the post type

* Updating gutenberg reference.

* Removed duplicate line from merge.

* updafe test files and iOS version for running locally with xcode 11

* Update gutenberg ref

* Update gutenberg ref

* Update aztec to version that support reversed and start on lists.

* Update Gutenberg to version where list settings are active in native.

* update block list check and capabilities

* replace double click on android

* fix block insertion timeout

* update branch with develop

* Update gutenberg ref

* Update package.json and JS bundle for 1.18.0 release

* Update Gutenberg

* Update Gutenberg

* update gutenberg ref

* Update GB-reference.

* Update release notes.

* revert caps to iPhone Simulator

* Update Gutenberg ref

* Update Release notes

* Update gutenberg/ reference

* Update bundles

* Update GB reference.

* Fix spacing

* Add static method to Media class to create instance using mimeType

* Add flag to track when appending multiple selected media items as blocks

* Introduce mediaSelectionCancelled method in WPAndroidGlueCode

* Set flag to append blocks when multiple = false is not respected

* Only use appendUploadMediaFiles plural version

* Update gutenberg ref

* Update to latest Gutenberg master

* Patch jsdom to implement Element.closest()

* Bring back changes on package.json from 1.7.1

* Add docstring to the function

* Return null as per https://dom.spec.whatwg.org/#dom-element-closest

* Update Aztec version.

* Update GB reference.

* Update gutenberg ref

* Update RELEASE-NOTES.txt

* Update GB reference

* Improve code block style

* Update GB reference.

* Update gutenberg ref

* Update Media mimeType truncation to use enum names

* Add Javadoc for mAppendsMultipleSelectedToSiblingBlocks flag

* Set appends to sibling blocks flag explicitly for all requests

* Add clarifying comment for special block append handling

* Remove singular (unused) appendMedia method

* Update GB reference.

* Update GB reference to master.

* Set appends to sibling blocks flag explicitly for other media pick

* Update GB reference.

* Update GB reference.

* Use lowercase for Media mimeType truncation

* Update Gutenberg

* Update Gutenberg

* Update Gutenberg ref - after fix for caption alignment

* Update bundles

* Point to aztecVersion hash which supports list with start and reverse attribute

* Update bundle and gutenberg ref - fix disappearing image

* Update GB reference.

* Update GB reference.

* Update GB reference.

* Update Gutenberg

* Update Gutenberg ref

* Update GB reference

* Update GB reference.

* Only enable page templates on dev builds

* Update aztec version

* Update aztec version

* Add colors for gallery block

* Update gutenberg reference

* Update gutenberg reference

* Update gutenberg reference

* Update GB reference.

* Update Gutenberg ref

* Update Aztec version to 1.14.0

* Make sure if the textColor is changed the default text color is updated.

* Use Slider from react-native-community lib (#1620)

* Use Slider from react-native-community lib

* Update slider version

* Add function to read npm version

* Rengenerate yarn.lock

* Bump node version

* Update readNPMVersion function

* Use Slider from fork

* Update Slider commit

* Use Slider from wordpress-mobile fork

* Add react-native-slider podspec

* Improve babel config

* Correct settings.gradle

* Correct project.pbxproj

* Update ref

* Bump

* Update ref

* Update ref to gutenberg master

* Update ref to gutenberg quick fix

* Update ref to gutenberg master

* Update Aztec editor version.

* Update to iOS 13

* Update Xcode version.

* Use iOS 12 for tests.

* Fix typo

* Update Aztec to fix CI error with xcode 10

* Use iOS 12.2

* Update Appium version.

* Update to appium 1.15.1

* Update to Appium 1.15.1 only in iOS

* Update caps.

* Add Gallery Block (#1498)

* WIP - initial-html.js for gallery testing

* Add parent app media mock for Android

* Update gutenberg reference

* Update gutenberg reference

* Comment out line setting mPendingMediaUploadCallback to null

* Update gutenberg reference

* Generate bundles

* Update gutenberg ref

* Generate bundles

* Generate bundles

* Add some color-studio colors for gallery

* Update gutenberg reference

* Generate bundle

* WIP update ref

* Update gutenberg reference

* Update gutenberg submodule

* Generate bundles

* Add $gray-40 color

* Update gutenberg reference

* Generate bundles

* Update gutenberg reference

* Generate bundles

* Update gutenberg reference

* Generate bundles

* Update gutenberg reference

* Generate bundles

* Update gutenberg reference

* Update gutenberg reference

* Update gutenberg reference

* Update gutenberg reference

* Update gutenberg reference

* Update gutenberg reference

* Update gutenberg reference

* Update gutenberg reference

* Restore demo content

* Restore anonymous implementation of GutenbergBridgeJS2Parent

* Update gutenberg reference

* Generate bundles

* Bump up Aztec version on iOS Example app

* Update gb ref

* Generate bundles

* Update gutenberg ref

* Update gutenberg ref (#1646)

* Add release notes for Gallery (#1658)

* Make sure we use iPhone 11 (iOS 13) for build and run tests

* Pooint aztecVersion to develop

* Update Appium for Android tests too.

* Set Appium to 1.15.0

* Update aztec version to 1.3.36

* Update appium to 1.16.0-rc.1

* Update Aztec iOS to 1.14.1

* Update GB reference.

* Activate preformat block on android platform (#1615)

* Updates package.json and JS bundle for 1.19.0 release.

* Update to shorten git commands

Make the git commands a little easier to copy by taking out `$` from the start of the lines.

This also matches with the other commands on the page which do not start with `$`.

* Update Unit Tests headings in Getting Started documentation

The heading "Test" should be "Unit Tests".

The heading "Writing and Running Tests" should be "Writing and Running Unit Tests".

https://github.com/wordpress-mobile/gutenberg-mobile/blob/develop/src/index.test.js

* Update gutenberg reference

* Update gutenberg reference

* Update gutenberg ref

* Updating bundles.

* Feat: Navigation Down in InnerBlocks (#1379)

* Add ref to gutenberg repo

* Fix e2e tests

* Update ref to gutenberg master

* Update Gutenberg ref

* Remove empty line between checkboxes

* Updating release notes to show video settings in 1.19

* Updating gutenberg reference to latest 1.19 release change on gutenberg master branch

* Update gutenberg ref

* Sass Transformer: Also look for partials

Add support for [SASS partials](https://sass-lang.com/guide), as is already present [upstream](https://github.com/kristerkari/react-native-sass-transformer/blob/52884dd59582856fa17e2b2e8ca9efc37d412387/index.js#L41-L42).

This should fix the issue introduced by #19159, and discussed there.

* Single quotes

* Also update sass-transformer-inside-gb.js

* Update release notes

* Update gutenberg ref

* Update gutenberg ref

* Update gutenberg ref

* Update gutenberg ref

* Update Gutenberg ref

* Update gutenberg ref

* Update gutenberg ref

* Bundles up to date with merged code from develop

* Add ref to gutenberg repo

* Update ref

* Update ref

* Update ref

* Upgrade the SVG lib to fix #1703

* Upgrade the Video lib to fix #1705

* Upgrade the Slider lib to use node_modules in local npm build

* Update ref

* Upgrade the SVG lib ref

* Upgrade the Video lib ref

* Upgrade the Slider lib ref

* Update ref

* Brings back gb master to normal (#1722)

* Fix/Bring back master to normal (#1724)

* Update Gutenberg ref

* Update Gutenberg ref

Co-authored-by: Tugdual de Kerviler <dekervit@gmail.com>
Co-authored-by: Matt Chowning <mchowning@gmail.com>
Co-authored-by: etoledom <etoledom@icloud.com>
Co-authored-by: Sérgio Estêvão <sergioestevao@gmail.com>
Co-authored-by: Cameron Voell <cameronvoell@gmail.com>
Co-authored-by: Jorge Bernal <jbernal@gmail.com>
Co-authored-by: Javon Davis <JavonDavis@users.noreply.github.com>
Co-authored-by: Maxime Biais <maxime@bia.is>
Co-authored-by: Gerardo Pacheco <gerardo.pacheco@automattic.com>
Co-authored-by: Matthew Kevins <mkevins@users.noreply.github.com>
Co-authored-by: Stefanos Togoulidis <stefanostogoulidis@gmail.com>
Co-authored-by: Marko Savic <savicmarko1985@gmail.com>
Co-authored-by: Luke Walczak <lukasz.walczak.pwr@gmail.com>
Co-authored-by: Pinar Olguc <pinarolguc@gmail.com>
Co-authored-by: Sheri Bigelow <sheri@designsimply.com>
Co-authored-by: jbinda <jakub.binda@gmail.com>
Co-authored-by: Bernie Reiter <ockham@raz.or.at>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gziolo gziolo gziolo approved these changes

@simison simison Awaiting requested review from simison

@sirreal sirreal Awaiting requested review from sirreal

Assignees

@ockham ockham

Labels

[Type] Bug An existing feature does not function as intended

Projects

None yet

Milestone

Gutenberg 7.2

Development

Successfully merging this pull request may close these issues.

5 participants

@ockham @simison @hypest @gziolo @youknowriad