-
Notifications
You must be signed in to change notification settings - Fork 598
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 beautiful property to control busy cursor during spawn #1613
Conversation
|
||
local app_starting = {} | ||
|
||
local cursor_waiting = "watch" | ||
|
||
--- Show busy mouse cursor during spawn. | ||
-- @beautiful beautiful.enable_spawn_cursor |
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.
+-- @beautiful beautiful.enable_spawn_cursor
+-- @tparam[opt=true] boolean enable_spawn_cursor
lib/awful/startup_notification.lua
Outdated
local function update_cursor() | ||
if #app_starting > 0 then | ||
if #app_starting > 0 and (beautiful.enable_spawn_cursor or beautiful.enable_spawn_cursor == nil) then |
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.
beautiful.enable_spawn_cursor ~= false
will do
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.
Sure! Too late and had too much work today. :)
Codecov Report
@@ Coverage Diff @@
## master #1613 +/- ##
==========================================
+ Coverage 80.66% 80.68% +0.02%
==========================================
Files 278 278
Lines 17353 17358 +5
==========================================
+ Hits 13997 14005 +8
+ Misses 3356 3353 -3
Continue to review full report at Codecov.
|
What about controlling the cursor being used (a default of "watch" then)? |
@blueyed I had the same idea but couldn't think of any other cursor that a user might want to use. That should be changed in the cursor theme. |
Why not, but I think this belong to it's own beautiful variables. Something like |
Will someone ever use another cursor than the current one or left_ptr? After thinking about it, I am not sure anybody will ever use such option. Did I miss something? Otherwise, if there is no further activity, I will merge this as-is in 2 days. |
No description provided.