-
debacle
How/where can I set the avatar image of a MUC in Gajim? `mod_vcard_muc.lua` is (should be) enabled.
-
debacle
After checking twice, I found it: Under "Manage"!
-
maximus
> gajim today suddendly broke the dark theme does anyone have an idea on what might be the cause?
-
maximus
also, does anyone know when and if gajim will ever support cross-client encrypted calls?
-
chud
There's already a functional pr just hanging there✎ -
chud
There's already a functional pr just hanging there nothing being done about it ✏
-
maximus
if it'll be merged into master, it'd be great
-
maximus
though I guess oss mainteinance is not that easy
-
cal0pteryx
In its current state, it cannot be merged into master.
-
cal0pteryx
(see the discussion at the merge request)
-
chud
There's one that can
-
nicoco
I see unresolved threads in both PRs…
-
cal0pteryx
chud, no, both cannot be merged into master (by discussion I meant to point to your MR)
-
chud
İ thought you just decided not to. But it can be merged, it is working.
-
taba
> chud, no, both cannot be merged into master (by discussion I meant to point to your MR) hasn't seen a response in a month... ↺
-
taba
the pr is 1 month old
-
cal0pteryx
chud: just because it's working doesn't mean it can be merged. It needs to fit conceptually, it needs to be maintainable, and it needs to be reviewed.
-
cal0pteryx
taba: what are you implying?
-
chud
Like the concept of working?
-
chud
Maintainable like what gajim currently has right
-
taba
> taba: what are you implying? how can it be unmergable if it hasn't seen a response in a month ↺
-
cal0pteryx
> Maintainable like what gajim currently has right We're working to make that better, not worse, right? ;) ↺
-
cal0pteryx
taba: please stop trolling
-
taba
oh my goodness...
-
cal0pteryx
taba: last warning
-
taba
i'm not trolling...
-
chud
>> Maintainable like what gajim currently has right > We're working to make that better, not worse, right? ;) Working would seem like an improvement over broken. But that's just me İ guess ↺
-
nicoco
I have much simpler and shorter PRs waiting for review for a lot more than that, so what? Volunteer maintainers set their priorities. Gajim is 20+ years old, it has a lot of technical debt, but AFAICT maintainers are working hard (unpaid!) to make sure every commit is the opportunity to improve the maintainability of the code base and set the bar high in terms of code quality and modern good practices.
-
maximus
exactly, I think we shouldn't demand or expect anything but instead be thankful to all of the devs working hard technically for free
-
chud
> to improve the maintainability of the code base and set the bar high in terms of code quality and modern good practices. Functionality not being one of those?
-
taba
> if it'll be merged into master, it'd be great maximus: fork it. will it get into a tagged version soon enough for it to matter to you? ↺
-
nicoco
Believe me, I understand your frustration chud, but aren't calls broken ATM because of poor maintainability of the previous implementation? FWIW nothing prevents you from running your commits on top of master. That's what I do with my MRs, and eating my dogfood makes me fix/improve things. The unresolved threads of the MR actually lists stuff to do for the MR to be accepted, don't give up, I'd be happy to have calls in back in gajim too.
-
chud
The last message from gajim maintainers says they'd go with the other branch, building up exactly on the previous implementation. İf you guys are actually willing to go with mine if the "code smell" of deep blocks is fixed, or anything of the kind, I'd be happy to oblige.
-
taba
removed by cal0pteryx
wrong chat
-
taba
wrong chat
-
taba
removed by cal0pteryx
wrong chat
-
a moderator
removed a message
wrong chat
-
a moderator
removed a message
wrong chat
-
taba
This person attempted to retract a previous message, but it's unsupported by your client.
-
chud
https://draugr.de/upload/00f530bb49b4e1685114d28f8bb6fb44b2fcbc2f/bzMibRCRfS2c0PgHW8YmnOtXjoy4IKcAeeUNsu0L/1000127240.jpg
-
chud
Lol
-
egnun
> egnun, what exactly makes gajim usable for server admins? lovetox, well because of all the admin and channel operator features, Gajim supports.✎ -
egnun
> egnun, what exactly makes gajim usable for server admins? lovetox, well because of all the admin and channel operator oriented features, Gajim supports. ✏
-
slichtzzz
Hi! lovetox, I'm sorry! I completely forgot about that Weblate translation engine ) So, I just pushed the translation right in.
-
slichtzzz
In fact I tried the service again and confirmation letter are not coming as always.
-
slichtzzz
So, if there's no workaround, it looks like I'm completely stuck with that comrimation-letter-not-coming issue
-
lovetox
slichtzzz, give me a moment
-
lovetox
i wrote our administrator again, i hope we can fix this email issue soon
-
MarsIronPI
Is there some tutorial for writing plugins, or should I just figure it out by looking at existing ones?
-
lovetox
you need to look at existing ones
-
MarsIronPI
ok
-
MarsIronPI
so, before I start, is it possible to write a plugin to support CSI?
-
lovetox
no
-
MarsIronPI
Oh? Plugins can't send raw stanzas?
-
lovetox
sorry i confused what CSI means
-
lovetox
should be possible to send a something on the stream
-
Maranda
client state indication lovetox ?
-
Maranda
https://xmpp.org/extensions/xep-0352.html
- Maranda thinks that's implemented already in Gajim
-
Maranda
IIRC
-
Maranda
even though that's more a mobile client thing rather than Desktop's.
-
lovetox
no its not, and i dont see it as really important for a desktop client
-
Maranda
👍️
-
maximus
> > if it'll be merged into master, it'd be great > maximus: fork it. will it get into a tagged version soon enough for it to matter to you? not sure what you mean here
-
lovetox
chud, as i wrote i see your MR as a proof of concept regarding the use of the libdatachannel. - The MR ignores that we have xmpp library - It provides no new framework for jingle related stuff - It does not integrate or tries to improve the current framework - it introduces new async concepts - in essence you dropped 5000 lines of code in separate modules which has no resemblence to the rest of the codebase still, i commend you for showing a working a/v solution with libdatachannel, and i will think about how we can salvage your work. Sadly the other contributor seems to be also not available anymore.
-
chud
One thing it does do is work tho
-
chud
As if the async stuff is a bad thing lol
-
chud
İ mean of course we can wait another year for glib guys to maybe merge their implementation✎ -
chud
İ mean of course we can wait another year(s?) for glib guys to maybe merge their implementation ✏
-
chud
Theirs will be superior, but it only *will* and mine is there and now and can deliver.
-
chud
Not sure what you mean by "ignores that we have xmpp library". Please tell me any specific issues that can at least in principle be fixed and I'll be happy to look into those. Like " doesn't display a user friendly message about a server not having tun/turn" yes İ can fix that. But "no semblance of the rest of the codebase" I'm not exactly sure what that means or how that is a breaking issue worth not having working calls.
-
chud
"framework"... Why do we want one, let alone why is it a dealbreaker? There's a framework currently and it's broken so what good is it. A more complex structure will be more fragile. Do calls and file transfers even share that much logic in practice?
-
chud
Also of those 5000 (is it 5000? Didn't count tbh, thought it'd be less) a good chunk are unit tests, which I'd argue will make it *more* maintainable.
-
lovetox
you use async concepts in your whole MR, this does not resemble the Gajim codebase, you did this without asking or discussing if this is something that this projects wants at all. seems you are comfortable with it so you went ahead. But are you there for the rest of the way? Converting the xmpp library and the rest of Gajim to async? i highly doubt it. you have a whole module dedicated to parsing and building xml stanzas, thats what a xmpp library is for, we work since years to get everything that is xml out of Gajim. We have now some abstraction for jingle, and why would i throw this away. Not working does not mean that the abstraction is wrong or not working, its simply unmaintained code that somewhere breaks and nobody had time to fix it. At least this is a major design decision which should be discussed before. I would have expected some kind of thoughts about how others after you can integrate and extend your code.<
-
lovetox
I dont see the vision of converting the codebase to async, i dont see any real benefit, there are sprinkled some points in the codebase where we need to request more than one IQ but thats it, thats for me not convincing enough to start adding async stuff.
-
debacle
lovetox MarsIronPI Maranda CSI might be useful even on a desktop client. E.g. when travelling with Deutsche Bahn or other train connections with poor internet connection, such as EuroStar or ... in fact most :-)
-
lovetox
debacle, you need a trigger so gajim tells the server the state
-
lovetox
on a phone this can be if the screen goes dark, or is locked
-
lovetox
bad wlan is not something gajim can detect and tell the server
-
debacle
lovetox Yes, on a desktop maybe "minimized" or not?
-
lovetox
yes or away
-
lovetox
but its not evident how a user would know he should minimize gajim, because his wlan is bad
-
lovetox
never heard of any application behaving that way before
-
lovetox
CSI implies that you are not active on the application
-
lovetox
because the server will not send you specific data anymore, like presence for example
-
debacle
lovetox Not likely, indeed. But at least if Gajim is minimized it would (maybe) eat less bandwidth or something.
-
lovetox
yes, CSI is for saving data when you are *not* active on the application
-
lovetox
its a "yeah why not" thing
-
lovetox
we would probably implement with the away timer
-
debacle
Sound right. Probably priority 563 of your long list ;-)
-
lovetox
yeah its definitly something i very much doubt a user would actively notice
-
lovetox
i would invest the time rather in things the user profits more from, like reactions, retractions
-
lovetox
file transfer, calls .. list is long
-
debacle
I fully agree!
-
lovetox
before i implement , 5% data save
-
debacle
Only saying: Just because an app is for desktop, it doesn't mean it were mobile, too.✎ -
debacle
Only saying: Just because an app is for desktop, it doesn't mean it weren't mobile, too. ✏
-
debacle
Too many negations, I'm confused by myself.
-
lovetox
slichtzzz, the admin of the server is on holiday until next week, so sorry, please be patient we will get it eventually
-
bot
lovetox pushed 1 commits to branch gajim/master imprv: Dismiss invitation notification when MUC is joined - https://dev.gajim.org/gajim/gajim/-/commit/bd7b61ed43e8b6e1e33641f15b87deffc0b472ca
-
MarsIronPI
lovetox: for me, CSI would be useful for running Gajim on my pinephone
-
MarsIronPI
lovetox: so plugins can't send the raw XML to the server?
-
lovetox
hm .. i think you can
-
MarsIronPI
Oh ok. Is there some existing plugin that does that I can look at?
-
MarsIronPI
lovetox?
-
chud
> you use async concepts in your whole MR, this does not resemble the Gajim codebase, you did this without asking or discussing if this is something t https://conference.gajim.org:5281/pastebin/9e6a5b18-8ea4-45d2-a817-6f148742617b
-
chud
My goodness
-
chud
Guess İ should've started that with "Dear lovetox, İ hope this letter finds you well"
-
chud
To sum up: I am more than willing to discuss and fix any issues with this, but please do give me feedback and specific issues you see so that I actually have something to reply to. Please let me know the condition of this being accepted into mainstream. I would love to devote more time to this, but I don't want it to be in vain.
-
chud
Btw one of the things that I personally would like to do is give user the option to choose codecs (e.g. on a shit connection you may wish to sacrifice opus quality for the narrowband pcmu )
-
fjklp
lovetox: I guess you already know but the fix for the slow startup worked well.
-
fjklp
fwiw, I do hope this a/v MR gets merged
-
MarsIronPI
fjklp: which one?
-
fjklp
I don't know the code of either and we having nothing working now so whichever has a chance :)
-
MarsIronPI
OK. I'm actually running one of them myself right now
-
lovetox
MarsIronPI, im not sure any plugin does that
-
MarsIronPI
lovetox: is there any API doc?
-
lovetox
no, look in the client.py module it sends stanzas
-
MarsIronPI
lovetox: OK, thanks!
-
lovetox
there is a nbxmpp client object there you need to access that directly to send non-stanzas
-
MarsIronPI
OK
-
fjklp
cal0pteryx: Damn, that new message highlighting is niiiiice. I see that we aren't scrolling now. Is this to prevent that bug of not getting the target message into view?
-
cal0pteryx
fjklp, I didn't touch scrolling, only how the animation is applied and the animation itself
-
MarsIronPI
What keys are required in the plugin-manifest.json?
-
fjklp
cal0pteryx: Hmm. On my flatpak, when I click on a message search result, the main chat window briefly shows the "home" screen (gajim icon and "Start Chatting" button), then displays the chat history and double-blinks the highlight.
-
cal0pteryx
fjklp, no clue, never saw that.
-
fjklp
are you using flatpak?
-
cal0pteryx
no, but that should not make a difference regarding message highlighting
-
fjklp
I'm seeing what I just described when clicking message search results and when I click reply links it simply pauses on the current screen, then immediately transitions the the place where the target message is, no scrolling animation.✎ -
fjklp
I'm seeing what I just described when clicking message search results and when I click reply links it simply pauses on the current screen, then immediately transitions the the place where the target message is, no scrolling animation. This is with flatpak. ✏
-
fjklp
I think it was always the case for me that flatpak did not show the highlight animation before these changes.
-
fjklp
I don't know why this would be unique to my machine
-
fjklp
I forgot that I set a custom theme. Flatpak is such a pain. I'll test it without the theme some other time.
-
fjklp
but I'm pretty sure flatpak behaved differently with animation with adwaita default
-
bot
wurstsalat pushed 1 commits to branch gajim/master refactor: Separate GroupChatInvitation widget from dialog - https://dev.gajim.org/gajim/gajim/-/commit/279227075bd641866b9a101a5ba9adbdcdd77dc6
-
bot
wurstsalat pushed 1 commits to branch gajim/master cfix: GroupChatInvitation: Provide accepted/declined signals and center widget - https://dev.gajim.org/gajim/gajim/-/commit/417c2b7c56280b3d1ea4be005952e9d94a6d0735