search

henrikruscon / hyper-statusline

Status Line Plugin for Hyper
MIT License
387 stars 79 forks source link

failed to load on windows #5

Closed luisrudge closed 7 years ago

luisrudge commented 7 years ago

image

henrikruscon commented 7 years ago

Wrote the code that fetches current CWD before Windows was on the map for Hyper. Currently relying on a UNIX exclusive command. I'm looking into viable alternatives right now. Pushing a fix soon! ✌️

Related https://github.com/henrikdahl/hyper-statusline/issues/4

luisrudge commented 7 years ago

maybe using https://www.npmjs.com/package/cwd?

luisrudge commented 7 years ago

I'm giving this a shot :)

luisrudge commented 7 years ago

yeah.. couldn't make this work on windows 😭

henrikruscon commented 7 years ago

No worries, I'm on it! Expect an update soon. ✌️

Naramsim commented 7 years ago

@luisrudge That means that in your ./hyper_plugins/node_modules hyper-statusline does not exist. You can npm i hyper-statusline in that folder, and the plugin will load. However, it does nothing 😭

I think something is broken in Hyperterm itself.

henrikruscon commented 7 years ago

@luisrudge Mind trying the os-windows branch to see if CWD works for you on Windows? I still can't debug myself ✌️

luisrudge commented 7 years ago

Ops.. Just saw your comment @henrikdahl. I'll test now

henrikruscon commented 7 years ago

@luisrudge Still needs work but since I can't debug myself it would be helpful to get some feedback during the process. Ideally the icons properly load now. New method for CWD fetching is a little flawed and needs improvement.

luisrudge commented 7 years ago

@henrikdahl ok. now it loads. But doesn't show up anything :( image

henrikruscon commented 7 years ago

@luisrudge Images are loading now, so that's an improvement. Seems the CWD method needs fixing. I'll look at it. Thanks ✌️