-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add imlib2 support for login screen customisation #1962
Conversation
Just re-read #1931. I've added an extra check so that wallpaper isn't loaded if the client has disabled wallpaper in the performance settings. |
Awesome. |
Overall, LGTM. |
AC_SUBST([IMLIB2_CFLAGS]) | ||
use_imlib2=yes | ||
esac | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Auto-detection is generally very nice but autodetected libraries will not appear in xrdp -v
. I want an easy way to tell users if your compiled xrdp supports png/jpg images on the login screen. ldd /usr/sbin/xrdp
sounds a little bit guru's way.
If we introduce more auto-detection, we might need to invent a new way to show which libraries/features compiled with.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's a really good point - I hadn't thought of that but it's a feature I use a lot when triaging problems. I'll remove the auto-detect.
I also noticed I'd not added CI support for i386 legacy. It's a small thing, but it's in now. |
I agree. |
Fixes #1929 #1819
See also discussion #1931
This PR allows for more customization of the xrdp login screen. An example customised screen from an Ubuntu 20.04 VM is:-
This is achieved by adding the following parameters to
xrdp.ini
:-File types other than bmp files are supported by way of the imlib2 library which is available on all our currently supported platforms. Imlib2 support is optional. Without it, we fall back to the existing bmp only code. The existing code has been reworked to (hopefully) make it simpler to understand, and also to fix #1819.
Alpha blending of images onto the background colour behind them is supported - this can be seen on the Ubuntu logo in the screenshot above. On multi monitor setups, the background image is displayed on the primary monitor only.
A test suite has been added for the extended
xrdp_bitmap_load()
call, so that the correct operation of the external library can be checked.This is a fairly significant PR, and so I suggest we hold off merging it until after the next release.