Skip to content

Commit

Permalink
Merge pull request #28 from eea/develop
Browse files Browse the repository at this point in the history
Apply privacy container height only on privacy check. Default to auto
  • Loading branch information
andreiggr authored Oct 19, 2022
2 parents 172ec2b + f35c58b commit d88b550
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 4 deletions.
7 changes: 6 additions & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,12 @@ All notable changes to this project will be documented in this file. Dates are d

Generated by [`auto-changelog`](https://github.com/CookPete/auto-changelog).

### [4.0.3](https://github.com/eea/volto-embed/compare/4.0.2...4.0.3) - 6 October 2022
### [4.0.4](https://github.com/eea/volto-embed/compare/4.0.3...4.0.4) - 19 October 2022

#### :hammer_and_wrench: Others

- Apply privacy container height only on privacy check. Default to auto [andreiggr - [`93a03fa`](https://github.com/eea/volto-embed/commit/93a03fa01f924d051c5e217f2ba2edf59cf58055)]
### [4.0.3](https://github.com/eea/volto-embed/compare/4.0.2...4.0.3) - 7 October 2022

#### :hammer_and_wrench: Others

Expand Down
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "@eeacms/volto-embed",
"version": "4.0.3",
"version": "4.0.4",
"description": "Embed external content",
"main": "src/index.js",
"author": "European Environment Agency: IDM2 A-Team",
Expand Down
3 changes: 1 addition & 2 deletions src/PrivacyProtection/PrivacyProtection.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -155,7 +155,6 @@ const PrivacyProtection = (props) => {
});
}
}, [enabled, url, path, dispatch, bgImg, show, intl, editable]);

return (
<VisibilitySensor
onChange={(isVisible) => {
Expand All @@ -169,7 +168,7 @@ const PrivacyProtection = (props) => {
className="privacy-protection-wrapper"
style={{
position: 'relative',
height: height ? `${height}px` : 'auto',
height: height && (!enabled || !show) ? `${height}px` : 'auto',
minHeight: '200px',
overflow: 'hidden',
}}
Expand Down

0 comments on commit d88b550

Please sign in to comment.