Yep, declaring constants for the strings is fine  - so long as code that tests e.g. `e.Show=='Hidden'` keeps working.

Oh, I was wrong, an Enum wouldn't do here. For some reason I was reading the diff backwards, I thought the change was from ints to strings, but it's the other way round.