websterParser / WebsterParser

Convert Webster's Unabridged 1913 dictionary in to a more usable format
GNU General Public License v3.0
351 stars 20 forks source link

Updates #11

Closed j-f1 closed 4 years ago

j-f1 commented 4 years ago

I’m happy to revert some of the more dramatic changes (TypeScript and ES6-ification specifically) if you’d like.

j-f1 commented 4 years ago

Also, a quick question: It looks like this repo is licensed under the GPL, but the package.json metadata says the license is ISC (which is like MIT). Which license do you intend to use?

jeffbyrnes commented 4 years ago

Can’t say about the licensing, but I do like this overall! I’m not well-versed in TypeScript, but I hear good things.

That said, it might be more beneficial to keep things simpler & not introduce TS? I’ll defer to @ponychicken as the maintainer for sure.

Also, #10 merged recently, so this needs a rebase.

j-f1 commented 4 years ago

Getting this error in the lint script:

Error Error: Received status code 403
    at IncomingMessage.res.on (/action/lib/request.js:13:25)
    at IncomingMessage.emit (events.js:203:15)
    at endReadableNT (_stream_readable.js:1145:12)
    at process._tickCallback (internal/process/next_tick.js:63:19)
{"message":"Resource not accessible by integration","documentation_url":"https://developer.github.com/v3/checks/runs/#create-a-check-run"}
ponychicken commented 4 years ago

The lint thing needs some love: https://github.com/hallee/eslint-action Only recently added support for .ts files, and right now we are using v1.0 which doesnt include this patch. But the master branch is also broken :( I guess we either fork & fix it, or take it out.

ponychicken commented 4 years ago

Bildschirmfoto 2020-05-20 um 08 59 26 Bildschirmfoto 2020-05-20 um 09 02 09

I added the custom font to fix the display of the accents (see screenshot). But maybe there are also other ways to fix it without adding a large font...

ponychicken commented 4 years ago

But the adjective part looks really nice!

jeffbyrnes commented 4 years ago

I think we can pull out the automated linting for now.

j-f1 commented 4 years ago

Weird, here’s what I’m seeing:

Screenshot_2020-05-20 08 11 06
jeffbyrnes commented 4 years ago

I will say, for me, I’ve really enjoyed that the Webster’s formatting is different. I.e., the red serif typeface for the entry titles.

j-f1 commented 4 years ago

I could try to add a setting in Dictionary preferences to switch the font.

Also, I tried to hide etymology from the dictionary popup you get by force clicking a word, but it seems to have stopped working.

ponychicken commented 4 years ago

Thanks – i merged most of you changes, except the CSS