Thanks to visit codestin.com
Credit goes to github.com

Skip to content

SKY: Implement intro/outro videos and stretch graphics in ibass - #7850

Open
Priyanshu3820 wants to merge 2 commits into
scummvm:masterfrom
Priyanshu3820:intro-impl
Open

SKY: Implement intro/outro videos and stretch graphics in ibass#7850
Priyanshu3820 wants to merge 2 commits into
scummvm:masterfrom
Priyanshu3820:intro-impl

Conversation

@Priyanshu3820

Copy link
Copy Markdown
Contributor

No description provided.

Comment thread engines/sky/control.cpp Outdated
_text->setSprite(NULL);
}
Common::Point mouse = _system->getEventManager()->getMousePos();
if (SkyEngine::isIbass()) {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

maybe something like void scaleCoordinates(Common::Point &point) which would be nop for non ibass would reduce the code duplication a little bit here?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

sure, good idea

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

done

@Priyanshu3820
Priyanshu3820 force-pushed the intro-impl branch 2 times, most recently from 8a6f88e to aec40d1 Compare August 19, 2026 13:27
@sev-

sev- commented Aug 24, 2026

Copy link
Copy Markdown
Member

Please rebase

@Priyanshu3820

Copy link
Copy Markdown
Contributor Author

Please rebase

done @sev-, but now since hotspots are implemented, they need scaling too along with the hint window. I will come back to it. Until then, please don't merge it.

@bluegr
bluegr marked this pull request as draft August 27, 2026 19:02
@Priyanshu3820

Copy link
Copy Markdown
Contributor Author

done, @sev-

@Priyanshu3820
Priyanshu3820 marked this pull request as ready for review August 31, 2026 08:15
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.

3 participants