search

rsms / fb-mac-messenger

⚡️ Mac app wrapping Facebook's Messenger for desktop
https://fbmacmessenger.rsms.me/
MIT License
2.86k stars 244 forks source link

UI bugs #407

Open alizahid opened 6 years ago

alizahid commented 6 years ago

I use Messenger reduced to one-third of my screen size, and I noticed a few bugs.

Broken typing indicator

screenshot 2017-08-13 18 52 17

Account information not visible (a messenger.com bug)

screenshot 2017-08-13 18 53 22

Settings icon missing entirely, and compose icon overlapping window buttons

screenshot 2017-08-13 18 56 38

A few of these have been reported before; I just wanted to consolidate.

mrleblanc101 commented 6 years ago

Just what I was going to report... It's sad that this app and Goofy (another messenger client) seems abandoned now

alizahid commented 6 years ago

I was using Messenger for Desktop, then tried Goofy for a few hours, and then tried this. Each have their share of bugs. So I just switched back to mobile for now.

hampustagerud commented 6 years ago

I have made a pull requests containing fixes for at least the compose icon. It's been a while though and I don't know if the project is abandoned or not. The settings icon is removed on purpose to give the app a more native look, you can access the settings as usual on macOS (CMD+, or Preferences in menu bar).

As you said the account information is a messenger bug but the typing indicator does not happen in the browser so that is a bug specific to this app I guess. However I have no idea how to fix it in the mess of HTML we get to hack here :) I would just make the window a tiny bit bigger for now.

amulya-b commented 6 years ago

There's a settings icon on the release I downloaded (latest one).

It's being overlapped by the default Mac window buttons.

shahzeb1 commented 6 years ago

@amulya-b could you post an image?

If you guys are down (@hampustagerud) we could fork this and hopefully maintain these UI fixes on our own? Because it seems like this project is semi-abandoned.

hampustagerud commented 6 years ago

@amulya-b My pull request is not merged and therefore not in any release yet.

@shahzeb1 I agree that this project does not seem to be very active anymore, which is too bad. I could maintain my fork with some help but that would require quite a lot from the new maintainers. We would need a new server for sparkle updates and everyone who is using this software would need to download the new release. Not to mention that all developers would need to know where to send pull requests.

Also I don't really feel comfortable just taking @rsms work and re releasing it as a new software without him actually giving it an OK or announcing that he is no longer maintaining his repo.

Another solution would be if @rsms added more maintainers because as far as I know he is the only one. This would require access to the sparlek server or he would still need to compile and upload every release. All current users would still be able to use their software and developers could just use the base repo.

This question is maybe better discussed as a new issue?