Improved Network, People, Chat & Mail page#119
Conversation
defnax
changed the title
Added to calculate background color when there is no Avatar available
Add Chat improvements Added Send Chat & Mail for Parcipants list Added tooltip functionality Improved Identiy details display Added show distant chat status
Fixed some issues on mails reply, replay all,forward Added sort function Added show per page mails
defnax
changed the title
|
Review? I will else continue with webui more |
|
Hey @defnax, Thanks for your patience, I need some time, will review it. |
Added statusbar
improve layout net settings Added missed config settings
|
i has one issue chat history scrollbar not working i dont know why, it does squeeze together, and going back and come again in the chat it shows only the last 20 messages, it does not shows all messages from current day @zelfroster |
Hey @defnax, I can take a look, but currently I am finding it difficult to compile RS on my mac. Do you have some idea about it? Can RS be compiled with qt-6? |
|
Normally its compileable with qt6 |
|
RetroShare compiles OK with cmake on linux/macos/win qt5/qt6 merge the following 6 PRs in master:
then read BUILD-cmake.md you can also contact me on RetroShare devs chat and forum |
|
Thanks defnax and jolavillette, I was able to compile with qt5 tho
@defnax the scrollbar works for me, i can scroll fine using mouse. and the reason it shows 20 messages is that the server is returning 20 messages only. 
Btw, I am looking at the webui after so long, and I think it can be improved a lot. |
|
But why you not using my pr? You showing me the master code |
My bad, I didn't build it again. Yeah I get the issue now It was a css issue, I have fixed it locally, will push it in a commit 
|
|
Thank you |
zelfroster
left a comment
zelfroster
left a comment
There was a problem hiding this comment.
Very good improvement overall, If we want to merge this soon, it will need at least the fixes which are not working.
I can push a few fixes here and there too.
There was a problem hiding this comment.
looks fine, but layout is a bit off
There was a problem hiding this comment.
Hmm wondering on my side looks nice, will look tonight
There was a problem hiding this comment.
look on my win 10 , maybe font or macos issue?
There was a problem hiding this comment.
looks fine, but layout is a bit off
seems your push changed someting the statusbar is not more black maybe you push changed something? it was fine till your changes
There was a problem hiding this comment.
I remember there used to be a green button for both tor and i2p socks proxy, we should enable it again, I see the code is there but not working from some reason.
There was a problem hiding this comment.
Hmm on my side i see grey but i doesnt tested yet with tor
There was a problem hiding this comment.
but there is no good api to get the status for tor and i2p i tested it not easy on qt it uses some sockets features
There was a problem hiding this comment.
i think it needs for tor status a api it not yet there , maybe need own pr, that gets more complex
|
Hi @csoler and @defnax, I have closed all the bot PRs by 64johnlee user, all those were created by AI without much context, changes were not tested, and some of them broke things instead of fixing. You can check more about it https://github.com/64johnlee/autopr. I believe using AI to accelerate work is great, but only when the author knows what they are doing and its tested/reviewed well. |
|
its possible fix scrollbar without destroy the styles code? |
3 participants
i created the changes with the help of gemini