Skip to content
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

Fixed code editor component height in logs layout (Management and Settings) #4253

Merged
merged 8 commits into from
Jun 17, 2022

Conversation

Machi3mfl
Copy link
Member

@Machi3mfl Machi3mfl commented Jun 13, 2022

Hi Team,
This PR fixes a Management > Logs layout to ensure that the "Load more logs" show correctly.

Closes #4206

@Machi3mfl Machi3mfl self-assigned this Jun 13, 2022
@Machi3mfl Machi3mfl added type/bug Bug issue UI/UX Generic label for things related to the font-end side backport 4.3-7.16 labels Jun 13, 2022
@Machi3mfl Machi3mfl requested a review from a team June 13, 2022 17:24
Copy link
Member

@Desvelao Desvelao left a comment

Choose a reason for hiding this comment

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

nitpick: add an entry to the changelog

Copy link
Member

@yenienserrano yenienserrano left a comment

Choose a reason for hiding this comment

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

Test: ✔️
CR: ✔️

2560x1440

image (8)

@chantal-kelm chantal-kelm self-requested a review June 16, 2022 14:43
@Machi3mfl Machi3mfl changed the title Fixed management logs layout Fixed height in logs layout Jun 16, 2022
@Machi3mfl Machi3mfl changed the title Fixed height in logs layout Fixed code editor component height in logs layout (Management and Settings) Jun 16, 2022
Copy link
Member

@chantal-kelm chantal-kelm left a comment

Choose a reason for hiding this comment

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

Tested in Chrome and Firefox browsers:

Test: ✔️
CR: ✔️

@github-actions
Copy link
Contributor

Jest Test Coverage % values
Statements 4.21% ( 1531 / 36375 )
Branches 1.79% ( 505 / 28242 )
Functions 3.29% ( 293 / 8896 )
Lines 4.25% ( 1478 / 34793 )

Copy link
Member

@AlexRuiz7 AlexRuiz7 left a comment

Choose a reason for hiding this comment

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

LGTM!

@AlexRuiz7 AlexRuiz7 merged commit b7eed0b into 4.3-7.10 Jun 17, 2022
@AlexRuiz7 AlexRuiz7 deleted the bugfix/4206-fixed-height-layour-management-logs branch June 17, 2022 11:02
github-actions bot pushed a commit that referenced this pull request Jun 17, 2022
…tings) (#4253)

* Fixed management logs layout

* Added dynamic offset

* Updated CHANGELOG

* Fixed offset calculation depends on height screen

* Used calc in overflow height in codeblock

* Added calc in settings logs

* changelog: replaced PR entry

Co-authored-by: Antonio David Gutiérrez <[email protected]>
(cherry picked from commit b7eed0b)
github-actions bot pushed a commit that referenced this pull request Jun 17, 2022
…tings) (#4253)

* Fixed management logs layout

* Added dynamic offset

* Updated CHANGELOG

* Fixed offset calculation depends on height screen

* Used calc in overflow height in codeblock

* Added calc in settings logs

* changelog: replaced PR entry

Co-authored-by: Antonio David Gutiérrez <[email protected]>
(cherry picked from commit b7eed0b)
AlexRuiz7 pushed a commit that referenced this pull request Jun 17, 2022
… (Management and Settings) (#4273)

Fixed code editor component height in logs layout (Management and Settings) (#4253)

* Fixed management logs layout

* Added dynamic offset

* Updated CHANGELOG

* Fixed offset calculation depends on height screen

* Used calc in overflow height in codeblock

* Added calc in settings logs

* changelog: replaced PR entry

Co-authored-by: Antonio David Gutiérrez <[email protected]>
(cherry picked from commit b7eed0b)

Co-authored-by: Maximiliano Ibarra <[email protected]>
AlexRuiz7 pushed a commit that referenced this pull request Jun 17, 2022
…out (Management and Settings) (#4274)

Fixed code editor component height in logs layout (Management and Settings) (#4253)

* Fixed management logs layout

* Added dynamic offset

* Updated CHANGELOG

* Fixed offset calculation depends on height screen

* Used calc in overflow height in codeblock

* Added calc in settings logs

* changelog: replaced PR entry

Co-authored-by: Antonio David Gutiérrez <[email protected]>
(cherry picked from commit b7eed0b)

Co-authored-by: Maximiliano Ibarra <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/bug Bug issue UI/UX Generic label for things related to the font-end side
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Management and Settings -> logs page: the load button appears half hidden
5 participants