-
Notifications
You must be signed in to change notification settings - Fork 3
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.
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
feat: Configure Images portlet into news page template - EXO-65514 - Meeds-io/MIPs#68 #917
Conversation
c90a091
to
9d15422
Compare
c6fce7c
to
ec14378
Compare
9d15422
to
efdf55e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
<container id="topNewsPage" template="system:/groovy/portal/webui/container/UIContainer.gtmpl"> | ||
<access-permissions>*:/platform/users</access-permissions> | ||
</container> | ||
<portlet-application> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think that we should rework the news template layout, to be verified with the PO
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think that the container id="newsPage" is nomore needed, even css classes already added not needed, to be verified with the final look
</portlet> | ||
<access-permissions>Everyone</access-permissions> | ||
<show-info-bar>false</show-info-bar> | ||
</portlet-application> | ||
<container id="newsBodyPage" template="system:/groovy/portal/webui/container/UIContainer.gtmpl"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<container id="newsBodyPage" template="system:/groovy/portal/webui/container/UIContainer.gtmpl"> | |
<container id="newsBodyPage" template="system:/groovy/portal/webui/container/UIResponsiveColumnGroupContainer.gtmpl.gtmpl"> |
</portlet> | ||
<access-permissions>Everyone</access-permissions> | ||
<show-info-bar>false</show-info-bar> | ||
</portlet-application> | ||
<container id="newsBodyPage" template="system:/groovy/portal/webui/container/UIContainer.gtmpl"> | ||
<access-permissions>Everyone</access-permissions> | ||
<container cssClass="newsBodyPageLeftColumn" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change newsBodyPageRightColumn and newsBodyPageLeftColumn from UIContainer.gtmpl to UISimpleColumnContainer.gtmpl
In that case I think that added css classes are no more needed
<container id="topNewsPage" template="system:/groovy/portal/webui/container/UIContainer.gtmpl"> | ||
<access-permissions>*:/platform/users</access-permissions> | ||
</container> | ||
<portlet-application> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think that the container id="newsPage" is nomore needed, even css classes already added not needed, to be verified with the final look
86b3609
to
bddce94
Compare
Your PR triggers too many exo-ci builds! Please finish your work and then, set your PR ready! Thank you |
53fc4bd
to
f3bc411
Compare
f3bc411
to
c298c24
Compare
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
efdf55e
to
096f107
Compare
No description provided.