Skip to content

Fix setBorderSizes() when called with 4 arguments#5325

Merged
SlySven merged 1 commit intoMudlet:developmentfrom
vadi2:fix-setborzersizes
Jul 19, 2021
Merged

Fix setBorderSizes() when called with 4 arguments#5325
SlySven merged 1 commit intoMudlet:developmentfrom
vadi2:fix-setborzersizes

Conversation

@vadi2
Copy link
Copy Markdown
Member

@vadi2 vadi2 commented Jul 19, 2021

Brief overview of PR changes/additions

Fix setBorderSizes() when called with 4 arguments

Motivation for adding to Mudlet

Bugfix

Other info (issues closed, discussion etc)

Fix #5324

Release post highlight

@vadi2 vadi2 requested a review from a team as a code owner July 19, 2021 15:08
@vadi2 vadi2 requested review from a team July 19, 2021 15:08
@add-deployment-links
Copy link
Copy Markdown

add-deployment-links bot commented Jul 19, 2021

Hey there! Thanks for helping Mudlet improve. 🌟

Test versions

You can directly test the changes here:

No need to install anything - just unzip and run.
Let us know if it works well, and if it doesn't, please give details.

Copy link
Copy Markdown
Member

@SlySven SlySven left a comment

Choose a reason for hiding this comment

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

Hey, that is exactly the change my PR would have done - if I could have uploaded it from my current location (my air-conditioned local authority's library building)!

<aside>I can't get ssh to work on this location in Windoze...! 🙄 </aside>

@github-actions
Copy link
Copy Markdown
Contributor

clang-tidy review says "All clean, LGTM! 👍"

@SlySven
Copy link
Copy Markdown
Member

SlySven commented Jul 19, 2021

FTR This bug is @Kebap's - from #4661.

@Kebap
Copy link
Copy Markdown
Contributor

Kebap commented Jul 19, 2021

FTR This bug is @Kebap's - from #4661.

Who's keeping record? 😲

@SlySven
Copy link
Copy Markdown
Member

SlySven commented Jul 19, 2021

Who's keeping record? 😲

No-one - which, given my record, is just as well! 😜

@SlySven SlySven merged commit 936c94e into Mudlet:development Jul 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

SetBorderSizes function error

5 participants