netrunner icon indicating copy to clipboard operation
netrunner copied to clipboard

Explicit reveals for jinja city grid

Open NBKelly opened this issue 2 years ago • 4 comments

Right now I've solved the "runner abilities don't appear to work" issue by having the dropdowns drop "down" when the ability belongs to a card in the 'corp-top-opponent' area.

The exact offset number (115%) can probably be tweaked for best aesthetics, I don't really have a good eye for that.

For now, Closes #6663.

example example3 example2

Next up will be

  1. implementing the ability for jinja
  2. maybe implementing it for corporate defector as well

NBKelly avatar Oct 04 '22 10:10 NBKelly

So as far as I can tell, this works.

  • There's a runner-ability on Jinja City Grid which allows the runner to say yes/no to explicit reveals
  • The value they choose is propagated across all active copies of Jinja city grid (no need to click another one if you happen to trash the one you chose)
  • it gets copied to a newly rezzed Jinja city grid

What it does:

  • If the runner says yes, then you get a little box popping up forcing you to read the cardname whenever the corporation reveals a card

runner-prompt

Why:

  • The physical act of revealing/showing a card is a big part of functional memory. Hiding that in a chat log, which is usually overwhelmed with information, strongly effects the power level of these cards (in my opinion).

Getting the runner prompt to not interact with the corp prompt/ability requires a little bit of an ugly hack. If there's a better way to do it, I would be happy to change it.

  • It can't complete the eid because that ruins things on the corp side
  • Making the prompt async and deliberately failing to complete the eid appears to work

NBKelly avatar Oct 05 '22 01:10 NBKelly

Still says WIP, so any updates on this?

NoahTheDuke avatar Dec 12 '22 14:12 NoahTheDuke

oh, this is fully functional, it's more a question of is that ugly hack I did going to cause any damage/other side-effects?

If no, then this is perfectly good to go.

NBKelly avatar Dec 12 '22 19:12 NBKelly

Okay, I never replied or merged this but I think that it's viable. I wonder if we could generalize this, maybe put it into reveal? That way we don't have to mess with eids.

NoahTheDuke avatar Nov 15 '23 15:11 NoahTheDuke

lmao i can't seem to merge this. you have commit power so you know what? merge at your pleasure. i trust you to get it right.

NoahTheDuke avatar Jun 28 '24 20:06 NoahTheDuke

I think I'm going to shelve this in favor of writing a direct 'explicit-reveal layer that just plugs into our reveal interface and uses a setting in the side pane.

NBKelly avatar Jun 28 '24 20:06 NBKelly