Closed webdev03 closed 2 years ago
Okay, conflicts?
Uncaught TypeError: Neofetch.osList.chrome.replaceAll is not a function
at Object.Neofetch.getData (neofetch.js:153:30)
at neofetch-js-ihad388h…s.vercel.app/:60:29
Broken...
Icon:
################ ################
################ ################
################ ################
################ ################
################ ################
################ ################
################ ################
################ ################
################ ################
################ ################
################ ################
################ ################
################ ################
################ ################
Icon:
.,coooooooooooooc,.
.,lllllllllllllllllllll,.
;ccccccccccccccccccccccccc;
'ccccccccccccccccccccccccccccc.
,ooc::::::::okO0000OOkkkkkkkkkkk:
.ooool;;;;:xK0kxxxxxk0XK0000000000.
:oooool;,;OKdddddddddddKX000000000d
lllllool;lNdllllllllllldNK000000000
llllllllloMdcccccccccccoWK000000000
;cllllllllXXc:::::::::c0X000000000d
.ccccllllllONkc;,,,;cxKK0000000000.
.cccccclllllxOOOOOOkxO0000000000;
.:cccccccclllllllloO0000000OOO,
,:ccccccccclllcd0000OOOOOOl.
'::cccccccccdOOOOOOOkx:.
..,::ccccxOOOkkko;.
..,:dOkxl:.
We didn't have your OS icon, so we used your browser icon instead.
Seems like you should've used if/else but used if instead.
@webdev03 Run git pull
, I made a commit
You can revert that commit now
Figured out the icon issue. The icons can't be indented. Going to fix.
Fixed an icon.
Looks good. Linux:
Merging, anything else?
Wait
Lines 117 and 118 both use deprecated navigator.platform
is there an alternative?
Let me see
Also, can I have some ASCII art to put for "Unknown"?
Yes, we could use navigator.userAgentData.platform
, but it'd require a major rewrite and every platform check would need changed. Also, we use the platform as the sole source of truth for both Windows and Linux. We could make it use the new version and fallback when necessary, but it'd take a lot of work. It'd be worth it, but it's definitely not a 2-minute change.
Also, can I have some ASCII art to put for "Unknown"?
Sure. Do a question mark.
Could you do a new PR for a question mark after this is merged, by chance? This one's a bit big.
Wait, some of the browser-specific code is missing question marks!
You didn't tell prettier to ignore the Windows icon
I got it
I fixed it
Wait, some of the browser-specific code is missing question marks!
?
They don't need question marks - navigator.userAgent
it's a standard right?
They don't need question marks -
navigator.userAgent
it's a standard right?
Not for Node. I'm trying to make it work in Node. If it will work, I can write tests easier.
I'll make it work for Node soon if you want. We can use TSUP to export to UMD/AMD/CJS/ESM
I'll make it work for Node soon if you want. We can use TSUP to export to UMD/AMD/CJS/ESM
No typescript! I just want to make it work no matter what! If it is designed to account for Node from the start, then at the most we'll just have to define navigator
if it's undefined. If it's not, it'll be hard.
Could you revert https://github.com/hello-smile6/neofetch-js/pull/9/commits/8c5847986a6f8de7090498124b703b842cb84015 ? I like the "Unknown" defaults!
The "Unknown" defaults will make Node support easier.
@webdev03 Fixed the defaults
Oh it's because Unknown hasn't been added to osList yet. If you can provide me the ASCII for Unknown then I can use it!
Oh it's because Unknown hasn't been added to osList yet. If you can provide me the ASCII for Unknown then I can use it!
Just use this for now. :)
?
Maybe I'll push an update later with a better question mark.
Just set formatOnPaste to false using vscode settings. That should help for those who have vscode configured to automatically format code.
@webdev03 Pull the changes
@webdev03 Can I merge? This is blocking ANSI support.
Sure
Okay!
Not tested yet on Chrome - can you tell me the results?