Skip to content

[RNMobile] Adjust new borders implementations to columns PR #21073

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.

Sign up for GitHub

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

Merged
jbinda merged 33 commits into from
Mar 26, 2020
Merged

[RNMobile] Adjust new borders implementations to columns PR #21073

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
33 commits
Select commit Hold shift + click to select a range
1acf841
first working version - w/o media text
dratwas Mar 10, 2020
ea78723
fix not visible borders
dratwas Mar 11, 2020
21f11c8
Merge branch 'master' into rnmobile/block-borders
dratwas Mar 12, 2020
db92a0c
TMP media-text
dratwas Mar 13, 2020
b337fb4
move inline styles to styles file
dratwas Mar 13, 2020
413e4a7
Working version with media text
dratwas Mar 13, 2020
cf924e8
Remove unused styles
dratwas Mar 16, 2020
cfca28e
fix toolbar styles
dratwas Mar 16, 2020
87e493a
use variable in floating toolbar style
dratwas Mar 16, 2020
5cc2190
fix cover block
dratwas Mar 16, 2020
89873ba
fix base variables
dratwas Mar 16, 2020
e14e8ca
add basic horizontal support
dratwas Mar 17, 2020
4268f57
change approach and use negative margins
dratwas Mar 19, 2020
69eec82
fix appender in empty group
dratwas Mar 19, 2020
67bb907
Merge branch 'rnmobile/block-borders' into rnmobile/column-block-borders
dratwas Mar 20, 2020
55c6e17
new approach of borders
dratwas Mar 23, 2020
a760d74
Fix appender
dratwas Mar 23, 2020
2f9ec61
fix toolbar margin
dratwas Mar 23, 2020
e966382
Merge branch 'master' into rnmobile/block-borders
dratwas Mar 24, 2020
39eb44a
add comment and move borders spacing val to consts
dratwas Mar 25, 2020
316f79d
fix media-text and remove unused props form withSelect in block
dratwas Mar 25, 2020
295c391
remove not needed overflow visible
dratwas Mar 25, 2020
9b27c32
Merge branch 'master' into rnmobile/block-borders
dratwas Mar 25, 2020
a1479e5
add style mock
dratwas Mar 25, 2020
9599f88
Merge remote-tracking branch 'origin/master' into rnmobile/column-block
jbinda Mar 25, 2020
f1b7c0f
Merge branch 'rnmobile/column-block' into rnmobile/column-block-borders
jbinda Mar 25, 2020
41fda77
Merge branch 'rnmobile/block-borders' into rnmobile/column-block-borders
jbinda Mar 25, 2020
be8b42c
remove block-mobile-toolbar
jbinda Mar 25, 2020
de9ad00
Merge branch 'rnmobile/column-block' into rnmobile/column-block-borders
jbinda Mar 25, 2020
58b1e39
Merge branch 'rnmobile/column-block' into rnmobile/column-block-borders
jbinda Mar 25, 2020
cba5e96
strech Column placeholder when parent is not selected
jbinda Mar 25, 2020
3047ea8
remove duplicate scss classes
jbinda Mar 26, 2020
7f6f069
fix crop borders on Android
jbinda Mar 26, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
add comment and move borders spacing val to consts
  • Loading branch information
@dratwas
dratwas committed Mar 25, 2020
commit 39eb44a058a27685188230db649a60c9dac1c733
2 changes: 2 additions & 0 deletions packages/base-styles/_variables.scss
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -83,6 +83,8 @@ $block-bg-padding--h: $block-side-ui-width + $block-side-ui-clearance; // paddin
$dimmed-opacity: 1;

$block-edge-to-content: 16px;
$solid-border-space: 12px;
$dashed-border-space: 6px;
$block-selected-margin: 3px;
$block-selected-border-width: 1px;
$block-selected-padding: 0;
Expand Down
14 changes: 7 additions & 7 deletions packages/block-editor/src/components/block-list/block.native.scss
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -65,21 +65,21 @@

.solidBorder {
position: absolute;
top: -12;
top: -$solid-border-space;
bottom: 0;
left: -12;
right: -12;
left: -$solid-border-space;
right: -$solid-border-space;
border-width: $block-selected-border-width;
border-radius: 4px;
border-style: solid;
}

.dashedBorder {
position: absolute;
top: -6;
bottom: -6;
left: -6;
right: -6;
top: -$dashed-border-space;
bottom: -$dashed-border-space;
left: -$dashed-border-space;
right: -$dashed-border-space;
border-width: $block-selected-border-width;
border-radius: 2px;
border-style: dashed;
Expand Down
1 change: 1 addition & 0 deletions packages/block-editor/src/components/block-list/index.native.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -111,6 +111,7 @@ export class BlockList extends Component {

const containerStyle = {
flex: isRootList ? 1 : 0,
// We set negative margin in the parent to remove the edge spacing between parent block and child block in ineer blocks
marginVertical: isRootList ? 0 : -marginVertical,
marginHorizontal: isRootList ? 0 : -marginHorizontal,
};
Expand Down