aboutsummaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
Diffstat (limited to 'src')
-rw-r--r--src/ChatWindow.cpp39
-rw-r--r--src/RoomNotificationsWindow.cpp3
-rw-r--r--src/Window.cpp15
-rw-r--r--src/WindowNotification.cpp4
-rw-r--r--src/main.cpp6
5 files changed, 36 insertions, 31 deletions
diff --git a/src/ChatWindow.cpp b/src/ChatWindow.cpp
index 9cc4ff1..9fc810b 100644
--- a/src/ChatWindow.cpp
+++ b/src/ChatWindow.cpp
@@ -184,8 +184,7 @@ namespace dchat
window->windowNotification->show("Room name has to be between 1 and 32 characters");
else
{
- auto room = window->rooms->createRoom(roomName);
- setCurrentRoom(room);
+ window->rooms->createRoom(roomName);
}
break;
}
@@ -384,7 +383,9 @@ namespace dchat
leftPanelChannels.attach(*roomButton, 0, roomCount, 1, 1);
++roomCount;
- RoomData *roomData = new RoomData { leftPanelUsersLayout, messageAreaLayout, roomButton };
+ uint numMessages = 0;
+ uint numUsers = 0;
+ RoomData *roomData = new RoomData { leftPanelUsersLayout, messageAreaLayout, numMessages, numUsers, roomButton };
roomDataById[*room->id] = roomData;
if(!currentRoom)
{
@@ -407,6 +408,7 @@ namespace dchat
messageAreaStack.set_visible_child(roomIdStr);
topbar->setTitle(room->name);
currentRoom = room;
+ assert(roomDataById.find(*room->id) != roomDataById.end());
currentRoomData = roomDataById[*room->id];
if(room->localUser)
{
@@ -433,20 +435,15 @@ namespace dchat
void ChatWindow::addMessage(const RoomAddMessageRequest &request)
{
- auto roomMessages = request.room->messages;
- RoomMessage *lastMessage = nullptr;
- if(!roomMessages.empty())
- lastMessage = &roomMessages.back();
-
- if(lastMessage && lastMessage->creator->publicKey == request.message.creator->publicKey)
+ if(request.prevMessage && request.prevMessage->creator->publicKey == request.message->creator->publicKey)
{
- int64_t msgTimeDiff = (int64_t)request.message.timestampSeconds - (int64_t)lastMessage->timestampSeconds;
+ int64_t msgTimeDiff = (int64_t)request.message->timestampSeconds - (int64_t)request.prevMessage->timestampSeconds;
if(msgTimeDiff <= MERGE_MESSAGE_TIMESTAMP_DIFF_SEC)
{
- auto message = messageById[lastMessage->id];
- message->text.set_text(message->text.get_text() + "\n" + request.message.text);
+ auto message = messageById[request.prevMessage->id];
+ message->text.set_text(message->text.get_text() + "\n" + request.message->text);
// Since messages that are sent withing a timeframe are combined, several message ids can refer to the same message
- messageById[request.message.id] = message;
+ messageById[request.message->id] = message;
if(!request.loadedFromCache && *request.room->id == *currentRoom->id)
{
currentRoomData->messageAreaLayout->queue_draw();
@@ -456,20 +453,20 @@ namespace dchat
}
}
- std::string userNickname = request.message.creator->nickname;
+ std::string userNickname = request.message->creator->nickname;
if(userNickname.empty())
userNickname = "Anonymous";
- ChatMessage *message = Gtk::manage(new ChatMessage(userNickname, request.message.text, request.message.timestampSeconds));
- if(request.message.creator->avatarUrl.empty())
+ ChatMessage *message = Gtk::manage(new ChatMessage(userNickname, request.message->text, request.message->timestampSeconds));
+ if(request.message->creator->avatarUrl.empty())
message->avatar.url = "https://discordemoji.com/assets/emoji/PeepoHide.png";
else
- message->avatar.url = request.message.creator->avatarUrl;
+ message->avatar.url = request.message->creator->avatarUrl;
message->set_valign(Gtk::Align::ALIGN_START);
message->set_hexpand(true);
message->show_all();
- messageById[request.message.id] = message;
+ messageById[request.message->id] = message;
RoomData *roomData = roomDataById[*request.room->id];
- roomData->messageAreaLayout->attach(*message, 0, roomMessages.size(), 1, 1);
+ roomData->messageAreaLayout->attach(*message, 0, roomData->numMessages++, 1, 1);
// TODO: When we get a message in the current room we scroll to the bottom, but this should only be done if we are not manually scrolling to view old messages
if(!request.loadedFromCache && *request.room->id == *currentRoom->id)
@@ -487,7 +484,7 @@ namespace dchat
username->get_style_context()->add_class("username-list-username");
request.user->userdata = username;
RoomData *roomData = roomDataById[*request.room->id];
- roomData->leftPanelUsersLayout->attach(*username, 0, request.room->userByPublicKey.size() - 1, 1, 1);
+ roomData->leftPanelUsersLayout->attach(*username, 0, roomData->numUsers++, 1, 1);
fprintf(stderr, "Added user %s\n", request.user->publicKey.toString().c_str());
if(roomData == currentRoomData)
@@ -555,4 +552,4 @@ namespace dchat
auto adj = messageArea.get_vadjustment();
adj->set_value(adj->get_upper());
}
-} \ No newline at end of file
+}
diff --git a/src/RoomNotificationsWindow.cpp b/src/RoomNotificationsWindow.cpp
index a491840..ba47836 100644
--- a/src/RoomNotificationsWindow.cpp
+++ b/src/RoomNotificationsWindow.cpp
@@ -48,6 +48,7 @@ namespace dchat
dialog.get_content_area()->pack_start(*Gtk::manage(new Gtk::Label(msg)));
dialog.add_button("Yes", Gtk::RESPONSE_YES);
dialog.add_button("No", Gtk::RESPONSE_NO);
+ dialog.show_all();
switch(dialog.run())
{
case Gtk::RESPONSE_YES:
@@ -102,4 +103,4 @@ namespace dchat
row[roomNotifications->userPublicKeyColumn] = userPublicKeyStr;
row[roomNotifications->messageColumn] = request.message;
}
-} \ No newline at end of file
+}
diff --git a/src/Window.cpp b/src/Window.cpp
index 255986a..8a6652e 100644
--- a/src/Window.cpp
+++ b/src/Window.cpp
@@ -41,7 +41,7 @@ namespace dchat
return result;
}
- Window::Window() :
+ Window::Window() :
chatWindow(this)
{
set_border_width(0);
@@ -230,12 +230,15 @@ namespace dchat
dispatcher.connect([this]()
{
- std::lock_guard<std::mutex> lock(dispatcherMutex);
- for(DispatcherHandler &dispatcherHandler : dispatcherHandlers)
+ std::lock_guard<std::recursive_mutex> lock(dispatcherMutex);
+ // We make a copy because dispatcherHandler below can call main loop again (for example when scrolling) which will call this function,
+ // and we want to prevent the same dispatcher from being called multiple times
+ auto handlersCopies = dispatcherHandlers;
+ dispatcherHandlers.clear();
+ for(DispatcherHandler &dispatcherHandler : handlersCopies)
{
dispatcherHandler();
}
- dispatcherHandlers.clear();
});
Rooms::connect(bootstrapNode->c_str(), 27130, roomCallbackFuncs);
@@ -280,7 +283,7 @@ namespace dchat
void Window::dispatchFunction(DispatcherHandler func)
{
- std::lock_guard<std::mutex> lock(dispatcherMutex);
+ std::lock_guard<std::recursive_mutex> lock(dispatcherMutex);
dispatcherHandlers.push_back(func);
dispatcher.emit();
}
@@ -380,4 +383,4 @@ namespace dchat
queue_draw();
return true;
}
-} \ No newline at end of file
+}
diff --git a/src/WindowNotification.cpp b/src/WindowNotification.cpp
index 9e17108..84365e9 100644
--- a/src/WindowNotification.cpp
+++ b/src/WindowNotification.cpp
@@ -29,7 +29,7 @@ namespace dchat
label.set_text(text);
set_visible(true);
revealer.set_reveal_child(true);
- int showTime = (int)std::max(1.0, (double)text.size() * 0.1) * 1000;
+ unsigned int showTime = (int)std::max(1.0, (double)text.size() * 0.1) * 1000;
hideTimer = Glib::signal_timeout().connect([this]
{
@@ -43,4 +43,4 @@ namespace dchat
return false;
}, showTime + revealer.get_transition_duration());
}
-} \ No newline at end of file
+}
diff --git a/src/main.cpp b/src/main.cpp
index f6b2ae6..b985d53 100644
--- a/src/main.cpp
+++ b/src/main.cpp
@@ -6,6 +6,10 @@
#include <giomm.h>
#endif
+// TODO: When creating a room, immediately switch to it.
+// TODO: When changing nickname or avatar, do not merge next message with previous one even if next message is sent right after the previous one.
+// We want to see user changes immediately.
+
static int setWindowCss(dchat::Window &window, Glib::RefPtr<Gtk::CssProvider> css)
{
auto ctx = window.get_style_context();
@@ -32,7 +36,7 @@ static int setWindowCss(dchat::Window &window, Glib::RefPtr<Gtk::CssProvider> cs
int main (int argc, char *argv[])
{
- auto app = Gtk::Application::create(argc, argv, "dec05eba.dchat");
+ auto app = Gtk::Application::create(argc, argv, "dec05eba.dchat", Gio::APPLICATION_NON_UNIQUE);
auto css = Gtk::CssProvider::create();
dchat::Window window;
if(setWindowCss(window, css) != 0)