Testresults: Include an pass/fail icon in the table of content as well about pickles HOT 9 CLOSED

I committed a first version of this as well. Downside: there's a noticeable run-time penalty.

from pickles.

I took a peek at the new changes. The placeholders for the icons appear, but not the images themselves? Did you have this problem too?

from pickles.

OK this is weird. I realized that I couldn't see the icons when the other fontawesome.* files were missing. No problem, I manually added them and the icons appeared. I then modified the project to add these files as resources (see latest commit) and had Pickles write them to the destination. When I did this, the icons disappeared! Very weird.

Long story short, it appears that writing the resources to the output directory from Pickles does something to them that causes them to be read in differently. I'm investigating.

All that being said, when the icons work they look great!

from pickles.

Maybe we should treat the fonts as separate content files rather than embedded resources?

from pickles.

Possibly. Since they're just byte streams this shouldn't be a problem
though. If we treat them as content files that means we have to distribute
them around with the varous runners. Not my favourite option. Hrm. this
will requite a bit of thought

On Wed, Nov 21, 2012 at 2:22 AM, dirkrombauts [email protected]:

Maybe we should treat the fonts as separate content files rather than
embedded resources?

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/15#issuecomment-10587771.

from pickles.

That being said, Visual Studio makes dealing with content files quite painless IMO.

from pickles.

Bingo - solved. Silly me was using a copy strategy aimed at text files, not at binary files. That's now fixed. So I vote we close this issue :-)

from pickles.

Agreed. I'll look at it Monday when I'm back in town

Sent from my iPad

On 2012-11-22, at 2:59 AM, dirkrombauts [email protected] wrote:

Bingo - solved. Silly me was using a copy strategy aimed at text files, not at binary files. That's now fixed. So I vote we close this issue :-)

—
Reply to this email directly or view it on GitHub.

from pickles.

Verified, better late than never :)

from pickles.