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

Spacecarp hologram for AIs #4012

Closed
wants to merge 5 commits into from
Closed

Spacecarp hologram for AIs #4012

wants to merge 5 commits into from

Conversation

Jordie0608
Copy link
Member

@GunHog
Copy link
Contributor

GunHog commented Jun 25, 2014

MORE 👍 👍 👍

@ChangelingRain
Copy link
Contributor

STOP STOP STOP
you need to make an actual hologram sprite and not just use the holocarp sprite, because the holocarp has scanlines; holograms have scanlines added to them on creation, not built-in.

@Jordie0608
Copy link
Member Author

efffffffort

@Giacom
Copy link
Contributor

Giacom commented Jun 26, 2014

Couldn't you have just done this, if getHologramIcon() returns an icon type?

holo_icon = icon('icons/mob/animal.dmi',"holocarp")

@Jordie0608
Copy link
Member Author

The scanlines on the normal holocarp sprite are a different speed and size to those applied onto hologram sprites; so yeah I guess, but it would've been weird.

@Giacom
Copy link
Contributor

Giacom commented Jun 26, 2014

It would freak people out because they won't know the difference!

@GunHog
Copy link
Contributor

GunHog commented Jun 26, 2014

Gia, that is a GOOD thing. That is what you WANT! 👍

<h2 class='date'>01 July 2014</h2>
<h3 class='author'>Ikarrus updated:</h3>
<ul class='changes bgimages16'>
<li class='wip'><B>Central Security has approved an upgrade of the Securitron line of bots to a newer model.</B><BR> Highlights include:</li>
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't think this is right.

@Jordie0608 Jordie0608 closed this Jul 10, 2014
@Jordie0608 Jordie0608 deleted the toomuchsnapforthisdeckofcards branch July 20, 2014 11:20
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.

None yet

4 participants