New Icons - #103
Conversation
|
RSI Diff Bot; head commit 21969e2 merging into e3640e3 Resources/Textures/_Funkystation/Interface/Misc/antag_icons.rsi
Resources/Textures/_Funkystation/Interface/Misc/job_icons.rsi
Resources/Textures/_Funkystation/Interface/Misc/job_icons_borders.rsi
|
|
One thing, weren't the dotted outline meant to show who knows the command manual? |
The dotted outline never really had any consistency in terms of its original usage. The Station AI got a dotted outline. Senior icons originally had a dotted outline before we changed them over to the new palette. It's been vaguely used to indicate "authority", "mindshield status", "expected to mentor", etc. with basically zero consistency. The "command training manual" interpretation is one I've never even heard of before. Given that pretty much every interpretation of the dotted borders is subjective and often entirely superfluous, I opted to remove it after discussing the matter with @Teasq and @Catazeus. Also I think the icons look a lot better without them. |
…better way on a different PR
YaraaraY
left a comment
There was a problem hiding this comment.
tested, looks good to me in-game. saw a couple mistakes(?) in the .ftl files and job.yml, should be good to merge after checking those out
* microopt offset works on my machine award * tweaks
* Update README.md with new links. (#2) * Update README.md with new links. * Update README.md with Princess-Cheeseballs' provided links. Co-authored-by: Princess Cheeseballs <66055347+Princess-Cheeseballs@users.noreply.github.com> * Removed an extra space. Co-authored-by: Princess Cheeseballs <66055347+Princess-Cheeseballs@users.noreply.github.com> * Update actions_changelog_rss.py with the new domain. * Update stylesheet.xsl * Update SECURITY.md --------- Co-authored-by: Princess Cheeseballs <66055347+Princess-Cheeseballs@users.noreply.github.com> * fix (#14) * Switch RT submodule to new repo (#19) * Remove toggle action from winter coat (#24) * i think i got them all * minor cleanup * init (#13) * Bugfix: Moproach/spider solution regeneration (#41) * moproach/spider solutions with recharge * mop mop mop roach * more straightforward moproach solution inheritance * Adds temperature protection to scarves (#47) * Adds temperature protection to scarves * YAML conventions say specific stuff is better at the bottom * Remove scream action, replace with emote action (#55) * no mroe screaming * if your angry and you know it scream with action grant! *clap clap* * emote action is more generic now * use flag instead * Revert "if your angry and you know it scream with action grant! *clap clap*" This reverts commit 7c40ee48 * screaming im screaming im screaming * reversed oops * Fix handheld grinders (#50) * fix * TEST!!!!!!!!!! * comment * what * im so silly * explosion!!!!! * i hate tests * those can be asserts too * i hate tests * Replace all uses of System.Random with IRobustRandom, resolve all System.Random warnings. (#32) * Remove all uses of System.Random * Use IRobustRandom in BinomialNumberSelector.Get * Remove remaining ToolRefinableSystem S.Random inst * Arcade sprite fixes & metadata cleanup (#40) * arcade sprite & metadata cleanup * 4 space meta.json * Bring cargo order menu to the front when selecting a new item * Bugfix: Remove infinite board dupe in television/telescreen construction graphs. (#42) * Fix telescreen/wallmount TV dupe * fix tagged icons * Initial commit (#3) * Fix blankline after games rules (#77) Annoyed me. * Fix illegible FileNotFoundError occurrences when git isn't in path. (#6) * Update git_helper.py * Update git_helper.py * Update git_helper.py * Make borgs not RR (#84) * Cargo integration tests update (#18) * Updated Cargo Integration Tests * usings * revert NoCargoOrderArbitrage * unneeded ; * . * review requests * review requests v2 * Move server_fans.ogg to a SoundCollection (#82) * quieter server fans * Restore individual parameter fields * Document Client, Server, and Shared files in folders starting with "K". Except microwaves. (#91) * document the entire owl * Apply suggestions from code review Co-authored-by: slarticodefast <161409025+slarticodefast@users.noreply.github.com> --------- Co-authored-by: slarticodefast <161409025+slarticodefast@users.noreply.github.com> * Can cancel orders from the console you placed it on (#11) * Make `InteractionTest` inherit from `GameTest` (#90) * Make InteractionTest inherit from GameTest * Convert MousetrapTest to use some GameTest features * Document stubbed Setup method * LoadTestMap now sets TestMap like CreateTestMap * Maybe fix? * Oh, derp * Fix approval section on order labels always being "Self" (#15) * Fixed label bug * review request * Add FixturesComponent to BaseWallmount (#81) * BUI template (#57) * Remove tasers from security locker fills (#67) * Ambient Music fixes and tweaks (#28) * reorganize yml * medical and engineering marker components and mass marking * bed * increase cooldown * did i leave that in * Renames all EntitySystem "ProtoMan" instances (#100) * Remove "ProtoMan" instances * guess we're caching gases now * Microoptimize Vector2i.Offset (#103) * microopt offset works on my machine award * tweaks * Fix event horizons using negative radius (#105) Not even valid why is it NEGATIVE just use ZERO! * Fix ItemGridPiece using invalid sprite bounds (#102) It inverted bottom and top. * Atmos category in the construction menu (#94) * commit * move on over --------- Co-authored-by: iaada <iaada@users.noreply.github.com> * Space Law Revision (#1) * Update SpaceLaw.xml * Update SLCrimeList.xml * Update SpaceLaw.xml to reflect AJCM-git's suggestion. * fixed (#128) * Directional arrow (#118) * arrow * yoink orginal arrow rsi * done * fixes * i just learnd about ignored components * Update Content.Client/DirectionalArrowIndicator/DirectionalArrowIndicatorComponent.cs Co-authored-by: ScarKy0 <106310278+ScarKy0@users.noreply.github.com> * customArrows * docs * more docs * teg and blue arrow * misc * fixes * remove color * imma dummy * meta * soon to obsolete * class not partial --------- Co-authored-by: ScarKy0 <106310278+ScarKy0@users.noreply.github.com> * Update RT to 277.2.0 (#138) * Refactor RecipeManager to use LINQ and PrototypesReloaded (#49) * small refactor RecipeManager and fix prototype reloading * wait... why is that here? * maybe? * Add section to link to space-wizards PR URL to PR template (#143) * Construction menu recipe history (#8) * Bugfix: Fix DrawDepth subfloor gap width, add gap for mapping subfloor layers (#70) * Hide artifact effects from spawnmenu (#149) * [FIX] Make slices of food fill out the space where the whole food used to be instead of dropping onto the floor (#160) * Add NU1900 to WarningsNotAsErrors (#171) * Removed duplicate function from solutions refactor in pricing system (#164) * Hide virtual items in cryoUI (#175) Co-authored-by: Pok <113675512+Pok27@users.noreply.github.com> * Add Integration test for ExpireIdCardComponent (#162) Co-authored-by: Josh Greenslade <joshua.greenslade@onyxinsight.com> Co-authored-by: SlamBamActionman <slambamactionman@gmail.com> * Remove Staff of Entrance from Magic Event Spell (#114) * fix atmosphere system protoman * Revert "Update README.md with new links. (#2)" This reverts commit dffabf7. * Revert "Switch RT submodule to new repo (#19)" This reverts commit 1000a7b. * Revert "Add section to link to space-wizards PR URL to PR template (#143)" This reverts commit 48201fc. --------- Co-authored-by: Rem <GitRem@grifton.lol> Co-authored-by: Princess Cheeseballs <66055347+Princess-Cheeseballs@users.noreply.github.com> Co-authored-by: riccardi48 <39341154+riccardi48@users.noreply.github.com> Co-authored-by: Tayrtahn <tayrtahn@gmail.com> Co-authored-by: Jessica M <jessica@jessicamaybe.com> Co-authored-by: Whatstone <166147148+whatston3@users.noreply.github.com> Co-authored-by: Minerva <218184747+mnva0@users.noreply.github.com> Co-authored-by: ScarKy0 <106310278+ScarKy0@users.noreply.github.com> Co-authored-by: chromiumboy <50505512+chromiumboy@users.noreply.github.com> Co-authored-by: metalgearsloth <31366439+metalgearsloth@users.noreply.github.com> Co-authored-by: Kyle Tyo <akikai297@gmail.com> Co-authored-by: Connor Huffine <chuffine@gmail.com> Co-authored-by: portfiend <109661617+portfiend@users.noreply.github.com> Co-authored-by: rumaks <me@rumaks.xyz> Co-authored-by: ArtisticRoomba <145879011+ArtisticRoomba@users.noreply.github.com> Co-authored-by: āda <ss.adasts@gmail.com> Co-authored-by: iaada <iaada@users.noreply.github.com> Co-authored-by: ThatGuyUSA <thatguyusa123@gmail.com> Co-authored-by: InsoPL <lukasz.lindert@protonmail.com> Co-authored-by: AndrewFenriz <78079974+AndrewFenriz@users.noreply.github.com> Co-authored-by: SlamBamActionman <83650252+SlamBamActionman@users.noreply.github.com> Co-authored-by: Vladislav Suchkov <20380250+murolem@users.noreply.github.com> Co-authored-by: korczoczek <danielkorczok@gmail.com> Co-authored-by: B_Kirill <153602297+B-Kirill@users.noreply.github.com> Co-authored-by: Pok <113675512+Pok27@users.noreply.github.com> Co-authored-by: JoshGreenslade <joshgreenslade@gmail.com> Co-authored-by: Josh Greenslade <joshua.greenslade@onyxinsight.com> Co-authored-by: SlamBamActionman <slambamactionman@gmail.com> Co-authored-by: Rami <ramialanbagy@gmail.com>
About the PR
This PR:
_Funkystationfor consistency and ease of use.Why / Balance
If the circumstances around where jobs are going change, this PR will be updated accordingly.
Technical details
All the new icons are filed under
Resources/Textures/_Funkystation/Interface/Misc, in three different RSIs:job_icons.rsi: Any icon that can be used for an ID card (including Syndicate and Zombie, as the Agent ID has access to them) goes here.antag_icons.rsi: Any icon used to indicate antagonist factions (Blood Cult, Revolution, etc.) that isn't already underjob_icons.rsigoes here.job_icons_borders.rsi: The mindshield and broken mindshield (the latter currently not implemented mechanically) sprites go here now, for consistency with Silicon crew indicators (which have a very similar form and function) in the main folders.References to the original RSIs have been replaced with references to the new RSIs where appropriate (with deprecated icons pointing to the original to avoid "missing file" errors). These old icons have also had their
allowSelectionvariables to false to prevent them from showing up in the Agent ID. Thejob.ymlfor wholly new icons is put underResources/Prototypes/_Funkystation/StatusIcon. They have their own localisation strings underResources/Locale/en-US/_Funkystation/job.Media
(Red arrows indicate an "effective replacement", where the job icon does not replace the old icon entirely, but is slated for a job that will replace the job tied to the icon it replaces. The icon replacing the "?" is for the Hospitality Director outlined in funky-station/docs#96. )
Requirements
License
MIT
Breaking changes
(Not sure how breaking these are. I'll remove them if they don't actually matter.)
Quartermaster.png, and should be changed to that fromQuarterMaster.pngwhen referencing the icon in the new RSI.SalvageSpecialist.png, and should be changed to that fromShaftMiner.pngwhen referencing the icon in the new RSI.Changelog
🆑