Make display_template look like a daterm

[BryceStevenWilley]

They serve the same function (tap to get more information, and they don't navigate you away from the page), so it makes sense to brand them the same.

I'm not quite used to it yet, but I do think I will with a bit of time. Also have to keep in mind that most users aren't coming back again and again, they use our side a few times and don't return.

Fixes #208

[nonprofittechy]

You're right it will take a bit to get used to, but I think it's the right change. Looks good to merge

[miabonardi]

Is this only supposed to work on collapse_template? I am seeing difference between collapse and display templates

[ekressmiller]

For us this comes out looking beautiful in Firefox:

But kind of hard to read in Chrome:

I think the chunky dashes underneath become more of an issue with longer stretches of text like in the template headers. I'll think about it/look into what we can do on our servers but just wanted to mention it!

[BryceStevenWilley]

Good spot, I agree that the Chrome one isn't very readable. You can use some CSS in your interview to adjust it:

.al_collapse_template a.al_toggle, .al_display_template a.al_toggle {
  text-underline-offset: 3px;
}

Here's a screenshot: Left is chrome, right is firefox.

There's also text-decoration-thickness. Not sure how widely supported both are, but it might be good to space out the Chrome one a bit in AL core. Annoying that the browser's have horrible readability defaults.

(I didn't check, but you might want to see what the terms look like in chrome. I'm pretty sure I used the same CSS as the terms used, and if you fix this CSS, you'll want to fix the term CSS as well).

[ekressmiller]

Thanks Bryce!