Skip to content

Commit

Permalink
release: version 3.0.1
Browse files Browse the repository at this point in the history
### Bug Fixes
- **Fix items getting cut off in Section Block on Mobile**: Fixes the regression caused by the last release that makes some items hidden in mobile that are children of Section block.
  • Loading branch information
HardeepAsrani authored Aug 19, 2024
2 parents 3ac12c1 + 8b23d17 commit c4aefba
Show file tree
Hide file tree
Showing 5 changed files with 31 additions and 26 deletions.
36 changes: 18 additions & 18 deletions composer.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 0 additions & 2 deletions src/blocks/blocks/section/editor.scss
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,6 @@ html[lang="ja"], html[lang="ko"], html[lang="zh"], html[lang="zh-Hans"], html[la
max-width: unset;
margin: inherit;
flex: 1;
overflow: hidden;

/* Add this to override Neve styles */
&[class*="has-dark-"] p, &[class*="has-light-"] p {
Expand All @@ -54,7 +53,6 @@ html[lang="ja"], html[lang="ko"], html[lang="zh"], html[lang="zh-Hans"], html[la

> .block-editor-inner-blocks {
flex: 1;
overflow: hidden;
}

.wp-block-themeisle-blocks-advanced-columns {
Expand Down
1 change: 0 additions & 1 deletion src/blocks/blocks/section/style.scss
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,6 @@ html[lang="ja"], html[lang="ko"], html[lang="zh"], html[lang="zh-Hans"], html[la
background: var( --background );
transition: 0.3s;
word-wrap: anywhere;
overflow: hidden;

&:hover {
background: var( --background-color-hover );
Expand Down
11 changes: 8 additions & 3 deletions src/onboarding/components/Finish.js
Original file line number Diff line number Diff line change
Expand Up @@ -23,11 +23,16 @@ const Finish = () => {
const [ email, setEmail ] = useState( window.otterOnboardingData?.userEmail );
const [ isLoading, setIsLoading ] = useState( false );

const { hasUserOptedin } = useSelect( select => {
const {
hasUserOptedin,
currentTheme
} = useSelect( select => {
const { get } = select( 'core/preferences' );
const { getCurrentTheme } = select( 'core' );

return {
hasUserOptedin: get( 'themeisle/otter-blocks', 'onboarding-optin' )
hasUserOptedin: get( 'themeisle/otter-blocks', 'onboarding-optin' ),
currentTheme: getCurrentTheme()?.template || getCurrentTheme()?.stylesheet
};
}, []);

Expand Down Expand Up @@ -58,7 +63,7 @@ const Finish = () => {
'Cache-Control': 'no-cache'
},
body: JSON.stringify({
slug: 'raft', // We need to see how we can make it work for any themes
slug: currentTheme,
site: window.otterOnboardingData.rootUrl,
email
})
Expand Down
7 changes: 5 additions & 2 deletions src/onboarding/components/steps/SiteInfo.js
Original file line number Diff line number Diff line change
Expand Up @@ -133,9 +133,12 @@ const SiteInfo = () => {

const blocks = select( 'core/block-editor' ).getBlocks();

let siteLogoBlock = findBlock( blocks, 'core/site-logo' );
const siteLogoBlock = findBlock( blocks, 'core/site-logo' );
const siteTitleBlock = findBlock( blocks, 'core/site-title' );

if ( siteLogoBlock ) {
// We only want to replace site logo with site title if site title is not present.
// In themes like Neve FSE, we have both site title and site logo.
if ( siteLogoBlock && ! siteTitleBlock ) {
replaceBlock( siteLogoBlock.clientId, wp.blocks.createBlock(
'core/site-title',
{
Expand Down

0 comments on commit c4aefba

Please sign in to comment.