Closed rschwiebert closed 10 months ago
Thanks for flagging this, @rschwiebert! I agree with your assessment. I'll aim to make the instructions more explicit in the next update.
I also had this issue. It took a long time to find out how to do this. I came to this bug report to find out how to make case sensitive searches. Would at least be nice to have case-sensistivity or something in the regex_flag description so you can find it when searching.
@radostyle what version are you using? In develop branch (soon to be 3.0) there's a side panel with help information for the flags.
Thanks for flagging @rschwiebert, thanks for the seconding @radostyle, and thanks for the preview @saulpw. For now, I've addressed this in f73c51e, now live:
Closing for now, but feel free to continue the conversation and/or propose better phrasing.
@jsvine Thanks!
Nice, Thanks!
On Sun, Dec 31, 2023 at 8:14 PM rschwiebert @.***> wrote:
@jsvine https://github.com/jsvine Thanks!
— Reply to this email directly, view it on GitHub https://github.com/jsvine/intro-to-visidata/issues/41#issuecomment-1873107887, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABFCZI3NC5FLTCAFM4RTADYMILYJAVCNFSM6AAAAAAV26GMTSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZTGEYDOOBYG4 . You are receiving this because you were mentioned.Message ID: @.***>
The docs say
It took me entirely too long to figure out that
regex_flags
is the relevant flag, and that is defaulting toI
. Honestly I'd never seen there.compile
options used before. And then it took a little bit longer to determine that configuring the setting= ''
and not=
was necessary.I think it'd be helpful to be explicit about it.