search

mnutt / davros

Personal file storage server
Apache License 2.0
298 stars 35 forks source link

Davros 0.17.0 broken on Edge #51

Closed ocdtrekkie closed 8 years ago

ocdtrekkie commented 8 years ago

Sadly nearly unusable for me, right now. :/ Menu has vanished, weird scrollbar on the side column.

davrosonedge

paulproteus commented 8 years ago

What is the question-mark pointing out, out of curiosity?

ocdtrekkie commented 8 years ago

There is usually a menu button up there. Which is, for one, the only place you can delete a file.

paulproteus commented 8 years ago

Ah hah, right, thanks, I forgot.

mnutt commented 8 years ago

Ahh thanks for pointing these out! I'm going to try to out some automated tests in place to catch these in the future.

On Jun 8, 2016, at 6:52 PM, Asheesh Laroia notifications@github.com wrote:

Ah hah, right, thanks, I forgot. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread.

paulproteus commented 8 years ago

BTW @mnutt there is heightened interest over the past month or two in automated tests for Sandstorm apps. So if your testing framework happens to be reusable (e.g. "copy-paste these things into your .travis.yml") then that could be neat.

Having said that, these IE/Edge bugs are catchable without invoking Sandstorm at all, so you might not need it.

Also consider that Sauce Labs has free-of-cost multi-browser Selenium service for open source projects.

ocdtrekkie commented 8 years ago

Any word on when this might be fixed?

As a useful note, while testing if this was fixed, I drag selected the top bar, and discovered that the menu button actually IS there, and does work. It's just invisible. So it is probably mostly/"just" an icon display issue?

mnutt commented 8 years ago

This should be fixed now in 0.18. The webfont-based icon set needed some extra css.

ocdtrekkie commented 8 years ago

Looks good to me.