Skip to content

Commit

Permalink
Issue #63 : update to electron 24
Browse files Browse the repository at this point in the history
  • Loading branch information
squalou committed May 24, 2023
1 parent 1776f93 commit 422ac06
Show file tree
Hide file tree
Showing 24 changed files with 1,791 additions and 1,742 deletions.
5 changes: 5 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,11 @@ An electron-base client for Google Hangouts Chat, since Google didn't see fit to

## CHANGELOG

### 5.24.19-1

- Update to electron 24 to support new functions added to google chat (quote reply among them). As a result, tray icon support gets worse. See https://github.com/squalou/google-chat-linux/issues/63
- Clean some outdated icons.

### 5.21.19-1

- https://github.com/squalou/google-chat-linux/issues/51 : manually set `NO_REDIRECT_URL` to solve login with custom OAuth SSO providers issues. Comma separated list of urls is accepted ! (see issue in github for more details)
Expand Down
46 changes: 46 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,12 @@
Tray has changed ... again
https://github.com/electron/electron/pull/36472
https://github.com/electron/electron/pull/36333
https://github.com/electron/electron/issues/36602

broken *again* on electron 22+

See below Changelog **5.24.19-1**

# google-chat-linux-git

An electron-base client for Google Hangouts Chat, since Google didn't see fit to provide one.
Expand Down Expand Up @@ -50,6 +59,43 @@ So, **to use electron's Wayland rendering** edit `/usr/share/applciations/google

See full [CHANGELOG](./CHANGELOG.md).

### 5.24.19-1

Update to electron 24.

**Why ?**

Some functionalities are blocked for older browsers, for instance "quoted-reply", which has finally arrived in google chat. (2023, hello guys, wake up)

**Bad news**

electron sucks with Tray ... again

gnome users will want to try https://extensions.gnome.org//extension/615/appindicator-support/

instead of Ubuntu Appindicators (if in use on the distro)

Note that ... looks like it's a crappy situation (again)
- Ubuntu Appindicator required for electron -> 21, wont work for 22+
- Appindicator and KStatusNotifierItem required for electron 22+

AND THEN AGAIN ! Double click must be used instead of single click,

and that shit is on Gnome only, and of course poor to no wayland support

Tray F***ng Icons still failing in 2023, not like it's been around since 28 years.


**Weird news** though : notification on application shortcut seems to work on Gnome ... only when app is launched from sources ! (`./google-chat-linux.sh`)

I'm probably definitively done with this electron nightmare.

Best solution is probably hte ArchLinux packagine approach : without electron embedded/packaged. go wonder.

I've added `google-chat-linux-nvm-launcher.sh` that uses nvm, uses `nvm use --lts`, then starts `./google-chat-linux/google-chat-linux.sh`,
it can be referenced in a local .desktop file, and it will work. That's hjow I personally launch it.


### 5.21.19-1

- https://github.com/squalou/google-chat-linux/issues/51 : manually set `NO_REDIRECT_URL` to solve login with custom OAuth SSO providers issues. Comma separated list of urls is accepted ! (see issue in github for more details)
Expand Down
Binary file removed assets/icon/chat-favicon-new-non-notif-256dp.png
Binary file not shown.
Binary file removed assets/icon/chat-favicon-new-notif-256dp.png
Binary file not shown.
Binary file removed assets/icon/chat-favicon-no-new-256dp.png
Binary file not shown.
Binary file removed assets/icon/chat-favicon-offline-256dp.png
Binary file not shown.
Binary file removed assets/icon/icon.png
Binary file not shown.
Binary file removed assets/icon/[email protected]
Binary file not shown.
Binary file removed build/icons/128.png
Binary file not shown.
Binary file removed build/icons/16.png
Binary file not shown.
Binary file removed build/icons/24.png
Binary file not shown.
Binary file modified build/icons/256.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file removed build/icons/32.png
Binary file not shown.
Binary file modified build/icons/48.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file removed build/icons/512.png
Binary file not shown.
Binary file modified build/icons/64.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
7 changes: 7 additions & 0 deletions google-chat-linux-nvm-launcher.sh
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
#!/bin/bash
export NVM_DIR="$HOME/.nvm"
[ -s "$NVM_DIR/nvm.sh" ] && \. "$NVM_DIR/nvm.sh" # This loads nvm
nvm use --lts
SCRIPT_DIR=$( cd -- "$( dirname -- "${BASH_SOURCE[0]}" )" &> /dev/null && pwd )
$SCRIPT_DIR/google-chat-linux.sh

2 changes: 1 addition & 1 deletion google-chat-linux.sh
Original file line number Diff line number Diff line change
Expand Up @@ -4,5 +4,5 @@ echo $SCRIPTPATH
export NODE_PATH="$SCRIPTPATH/node_modules/.bin"
export PATH=$NODE_PATH:$PATH
# GTK_USE_PORTAL=1 is set from index.js
${NODE_PATH}/electron "${SCRIPTPATH}/src/index.js" --trace-warnings --ozone-platform-hint=auto
${NODE_PATH}/electron "${SCRIPTPATH}/src/index.js" --trace-warnings

Loading

0 comments on commit 422ac06

Please sign in to comment.