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 9-sliced background to chat #194

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

luk3yx
Copy link
Member

@luk3yx luk3yx commented Jun 27, 2024

Add compact, short information about your PR for easier understanding:

  • Goal of the PR
  • How does the PR work?
  • Does it resolve any reported issue?
  • If not a bug fix, why is this PR needed? What usecases does it solve?

To do

This PR is a Work in Progress / Ready for Review.

  • List
  • Things
  • To do

How to test

@MoNTE48
Copy link
Member

MoNTE48 commented Jun 27, 2024

Can you add screenshot how it looks when bg_common.png is used?

@luk3yx
Copy link
Member Author

luk3yx commented Jun 27, 2024

image

@MoNTE48
Copy link
Member

MoNTE48 commented Jun 27, 2024

Hmm, I think the console should be a little longer so as not to cut off the scrollbar.

@@ -348,7 +348,7 @@ void GUIChatConsole::drawBackground()
draw2DImage9Slice(
driver,
m_background,
core::rect<s32>(0, 0, m_screensize.X, m_height),
core::rect<s32>(0, 0, m_screensize.X, m_height + 16),
Copy link
Member

Choose a reason for hiding this comment

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

Well it just made it worse and cropped the background.

The comment is worded badly because I am tired, sorry.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants