• Printing errno on error would be very useful to debug socket communications.
• Implement an error message for cases when get_monitors fails (WM doesn't support _NET_DESKTOP_VEWPORT)
• Error handling in daemon
• Error handling in dexpo
• Handle xcb errors with xcb_generic_error_t

Memory usage is three times higher than it should be.
Remove redundant copying to fix.

Use xcb to create a window with customizable background and draw screenshot of current desktop there with a border and caption.

• Screenshot should be made at program start (in the next version screenshots will be fetched from daemon)
• Caption should be equal to _NET_DESKTOP_NAMES

Similar to other pagers

preview

There may be a use case when user needs to move all windows from one workspace to another.
Sounds like it is possible to implement.

Rieman

• Analyze the way it handles window moves and other events
• Analyze the way it parses viewports and desktop geometries (really bad at a glance)
• Analyze the way it draws fonts (UTF characters don't get drawn so idk)

Analyze other pagers and report below

This would simplify error handling

Add more customization options to dexpo's location:
Prepend - to the coordinate to calculate it from the left

The daemon has to

• Make and store Pixmaps of each virtual desktop together with the _NET_DESKTOP_NAME
• Send Pixmaps to GUI client over unix sockets
  • Make a socket server (daemon) and a socket client (GUI)

Old tests are obsolete now, so new have to be written.

Please add following cases:

• Can send and receive many pixmaps (with a benchmark)
• Test race conditions

Run sanitizers and profile the binary

in daemon.cpp:

desktops[current].save_screen ();
// Copying pixmap from desktops into socket pixmaps
socket_desktops[current].pixmap = desktops[current].pixmap;

Here pixmaps are copied to prevent race conditions.

Avoid copying and instead use mutex to prevent race conditions.

Overloaded operator[][] would be very handy when working with 1d pixmaps

The window has to draw focus on start and receive keyboard events.

• <Esc> to close window
• Arrow keys and hjkl to move preselection of desktop
• Enter to open the desktop (execute dummy code for now)

• Check with clang-format
• Check with clang-tidy
• Run include-what-you-use
• Run ctest

Currently cmake builds all shared .cpp files twice, as there are two executables in CMakeLists.txt.
This should be fixed

Based on my interactions with Linux applications, current GUI UX doesn't feel right.
Here is the list of what I would expect from it. You may disagree with me, so please respond with your expectations.

• Exit dexpo after desktop change event. (Why would I need dexpo running after I have moved to another desktop)
• Display preselection around current _NET_WM_DESKTOP on startup
• Draw border around all desktops
• Border size should not obstruct padding size

And add these options to the config file

Currently launching a second daemon process causes the crash of both.

It would be better if the second startup didn't affect the the first.

You can see the rightmost background part is one pixel too big

Same happens with vertical stacking, except then its the lower part that is too big

Codebase is becoming more intertwined and shitty so some action is required.

A major new feature would be a config parser because I don't like recompiling every time I change the config.

It would be more flexible to specify key name in config instead of key code.

Binding keys by their names and not keycodes would really help this

This is a minimalistic application and it needs a fitting name.

Pros:

• Less keystrokes to type name in the terminal
• dxpd (dexpo daemon) sounds cool

Cons:

• muh consistency

To rename or to keep? Discuss here

Writing

cmake -B ./build/
cmake --build ./build/

every time is tedious

Store data of a desktop recently returned by get_hover_desktop()

This function may be ran hundreds of times and each time it iterates over vector of desktops to find desktop's x and y that probably haven't changed.

Need to research whether xcb_ewmh will be better for this application

Dexpo havs many places that need debugging output.
Please add verbosity toggle to config and a sleek function to print to file/cerr

Test get_desktops() by sending sample EWMH atoms as WMs would do.

These atom values can be checked by hand in different WMs and added to tests.

Screenshots look crappy, maybe implementing some filter will fix this

• #48
• Desktops are missing one pixel on the left
• Desktop layouts are missing one pixel on the left

• Overlay a thin border around application windows on a screenshot
• Highlight window on hover/preselect
• Focus window on click
• Implement a way to traverse windows with keyboard shortcuts

Current setup makes screenshot based on a periodic timer. This can be changed to fit workflows better and be more optimized

Should be possible. dunst has same functionality

My guess is #30 has to be closed first

Need to sketch this look first

If a user has multiple monitors, launch a separate dexpo window for each of those monitors's desktops.

Not sure yet how to configure windows' locations.

Colors in the config do not support alpha channel.

Maybe adding some mask to the graphic context would fix this

• Rename classes and files to be more verbose
• Custom deleter for unique_ptr<xcb_get_image_reply_t> and others
• Consistent integer types
• Disable desktop names functionality for now
• Change #define to constexpr
• Comments in the doxygen style
• Move EWMH related functions from daemon.cpp to ewmh.cpp
• #21

Also dimensions of xmonad desktops can be changed during runtime, so this has to be handled properly.
Perhaps desktop initialization during runtime is unnecessary and should be done just before the screenshot is made.