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

Add return type to all util functions #650

Merged
merged 1 commit into from
Feb 16, 2024

Conversation

edlerd
Copy link
Collaborator

@edlerd edlerd commented Feb 15, 2024

Done

  • Add return type to all util functions

@webteam-app
Copy link

Demo starting at https://lxd-ui-650.demos.haus

@edlerd edlerd marked this pull request as ready for review February 15, 2024 17:46
@@ -259,29 +264,3 @@ export const getAbsoluteHeightBelow = (belowId: string) => {
parseFloat(style.paddingTop) + parseFloat(style.paddingBottom);
return element.offsetHeight + margin + padding + 1;
};

export const getElementAbsoluteHeight = (element: HTMLElement | null) => {
Copy link
Collaborator

Choose a reason for hiding this comment

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

sorry about these orphan functions!

@mas-who
Copy link
Collaborator

mas-who commented Feb 16, 2024

LGTM.

@edlerd edlerd merged commit fb0212b into canonical:main Feb 16, 2024
8 checks passed
github-actions bot pushed a commit that referenced this pull request Feb 16, 2024
@edlerd edlerd deleted the util-return-types branch February 16, 2024 09:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants