search

NEAR-Edu / near-certification-tools

2 stars 2 forks source link

fix: rename 404 text #45

Closed norrec99 closed 2 years ago

norrec99 commented 2 years ago

https://github.com/NEAR-Edu/near-certification-tools/pull/42#issuecomment-1061871395 I used capital letters for the replacement of "404" text

Here how it looks image

And this is the lower case "Error" message image

I think the first one looks better but I can change according to your liking

ryancwalsh commented 2 years ago

@norrec99 Of those 2, I prefer the second one ("Error"). But I'd also be curious to see a version where you remove "Error" (and the gray vertical line) entirely.

norrec99 commented 2 years ago

It looks like that:

image

ryancwalsh commented 2 years ago

@norrec99 Cool, I like that version (https://github.com/NEAR-Edu/near-certification-tools/pull/45#issuecomment-1064828394) the best.