search

theDavidCoen / LightningExchanges

List of exchanges that support Lightning Network
190 stars 48 forks source link

ReadMe.md: try reducing horizontal scroll #49

Closed knocte closed 12 months ago

knocte commented 1 year ago

Shortened some words (or separated them so that they can fit more nicely in next line) in order to avoid having to use horizontal scroll to see the table. Also merged 2 cols into 1.

You can see the result here: https://github.com/knocte/LightningExchanges/tree/patch-1

theDavidCoen commented 1 year ago

@knocte That's nice but unfortunately the need to use horizontal scroll depends on the monitor resolution / Browser zoom. See this screenshots from your fork: Screenshot from 2023-06-30 12-54-07

and

Screenshot from 2023-06-30 12-57-17

knocte commented 1 year ago

I'm not saying my solution is perfect, but it improves the situation.

theDavidCoen commented 1 year ago

@knocte I like this PR, not because of the UX but because of the SEO. In fact, it seems that your solution might fit better and give more info in Google Search, see: Screenshot from 2023-06-30 13-23-29

However, for SEO, I believe it's better if we keep the words as before, and use your solution to merge col 3 into col 2.

So I ask you if you can keep the column merge, but restore the wording, and I'll merge the PR.

knocte commented 12 months ago

@theDavidCoen done