more debug logging

previously activating a control meant tapping it once to enable
proximity, then tapping it again to do the actual interaction.
now a single tap will do that.

previously the following glitch was present:
- interact with a slider
- release touch on that slider
- touch a different control
->input would be received by the old control

now the input is received by the correct control

README.md

@@ -43,11 +43,10 @@ Most notable features:
     On Linux and macOS these terminal commands can be used to install or update uosc (if wget and unzip are installed):
 
     ```sh
-    config_dir="${XDG_CONFIG_HOME:-~/.config}"
-    mkdir -pv "$config_dir"/mpv/script-opts/
-    rm -rf "$config_dir"/mpv/scripts/uosc_shared
+    mkdir -pv ~/.config/mpv/script-opts/
+    rm -rf ~/.config/mpv/scripts/uosc_shared
     wget -P /tmp/ https://github.com/tomasklaen/uosc/releases/latest/download/uosc.zip
-    unzip -od "$config_dir"/mpv/ /tmp/uosc.zip
+    unzip -od ~/.config/mpv/ /tmp/uosc.zip
     rm -fv /tmp/uosc.zip
     ```
 

script-opts/uosc.conf

@@ -47,7 +47,7 @@ timeline_cache=yes
 #   `cycle:{default_icon}:{prop}[@{owner}]:{value1}[={icon1}][!]/{valueN}[={iconN}][!]`
 #     - button that cycles mpv property between values, each optionally having different icon and active flag
 #     - presence of `!` at the end will style the button as active
-#     - `{owner}` is the name of a script that manages this property if any
+#     - `{owner}` is the name of a scrip that manages this property if any
 #   `gap[:{scale}]` - display an empty gap, {scale} - factor of controls_size, default: 0.3
 #   `space` - fills all available space between previous and next item, useful to align items to the right
 #           - multiple spaces divide the available space among themselves, which can be used for centering
@@ -157,7 +157,7 @@ font_scale=1
 # Border of text and icons when drawn directly on top of video
 text_border=1.2
 # Use a faster estimation method instead of accurate measurement
-# setting this to `no` might have a noticeable impact on performance, especially in large menus.
+# setting this to `no` might have a noticable impact on performance, especially in large menus.
 text_width_estimation=yes
 # Execute command for background clicks shorter than this number of milliseconds, 0 to disable
 # Execution always waits for `input-doubleclick-time` to filter out double-clicks

scripts/uosc/elements/Menu.lua

@@ -106,7 +106,8 @@ function Menu:init(data, callback, opts)
 	self.key_bindings = {}
 	self.is_being_replaced = false
 	self.is_closing, self.is_closed = false, false
-	self.drag_last_y = nil
+	---@type {y: integer, time: number}[]
+	self.drag_data = nil
 	self.is_dragging = false
 
 	self:update(data)
@@ -544,37 +545,47 @@ function Menu:on_prop_fullormaxed() self:update_content_dimensions() end
 
 function Menu:handle_cursor_down()
 	if self.proximity_raw == 0 then
-		self.drag_last_y = cursor.y
+		self.drag_data = {{y = cursor.y, time = mp.get_time()}}
 		self.current.fling = nil
 	else
 		self:close()
 	end
 end
 
+function Menu:fling_distance()
+	local first, last = self.drag_data[1], self.drag_data[#self.drag_data]
+	if mp.get_time() - last.time > 0.05 then return 0 end
+	for i = #self.drag_data - 1, 1, -1 do
+		local drag = self.drag_data[i]
+		if last.time - drag.time > 0.03 then return ((drag.y - last.y) / ((last.time - drag.time) / 0.03)) * 10 end
+	end
+	return #self.drag_data < 2 and 0 or ((first.y - last.y) / ((first.time - last.time) / 0.03)) * 10
+end
+
 function Menu:handle_cursor_up()
-	if self.proximity_raw == 0 and self.drag_last_y and not self.is_dragging then
+	if self.proximity_raw == 0 and self.drag_data and not self.is_dragging then
 		self:open_selected_item({preselect_first_item = false, keep_open = self.modifiers and self.modifiers.shift})
 	end
 	if self.is_dragging then
-		local distance = cursor.get_velocity().y / -3
+		local distance = self:fling_distance()
 		if math.abs(distance) > 50 then
 			self.current.fling = {
-				y = self.current.scroll_y, distance = distance, time = cursor.history:head().time,
+				y = self.current.scroll_y, distance = distance, time = self.drag_data[#self.drag_data].time,
 				easing = ease_out_quart, duration = 0.5, update_cursor = true,
 			}
 		end
 	end
 	self.is_dragging = false
-	self.drag_last_y = nil
+	self.drag_data = nil
 end
 
 function Menu:on_global_mouse_move()
 	self.mouse_nav = true
-	if self.drag_last_y then
-		self.is_dragging = self.is_dragging or math.abs(cursor.y - self.drag_last_y) >= 10
-		local distance = self.drag_last_y - cursor.y
+	if self.drag_data then
+		self.is_dragging = self.is_dragging or math.abs(cursor.y - self.drag_data[1].y) >= 10
+		local distance = self.drag_data[#self.drag_data].y - cursor.y
 		if distance ~= 0 then self:set_scroll_by(distance) end
-		if self.is_dragging then self.drag_last_y = cursor.y end
+		self.drag_data[#self.drag_data + 1] = {y = cursor.y, time = mp.get_time()}
 	end
 	request_render()
 end
@@ -651,34 +662,26 @@ function Menu:disable_key_bindings()
 	self.key_bindings = {}
 end
 
--- Wraps a function so that it won't run if menu is closing or closed.
----@param fn function()
-function Menu:create_action(fn)
-	return function()
-		if not self.is_closing and not self.is_closed then fn() end
-	end
-end
-
 ---@param modifiers Modifiers
 function Menu:create_modified_mbtn_left_handler(modifiers)
-	return self:create_action(function()
+	return function()
 		self.mouse_nav = true
 		self.modifiers = modifiers
 		self:handle_cursor_down()
 		self:handle_cursor_up()
 		self.modifiers = nil
-	end)
+	end
 end
 
 ---@param name string
 ---@param modifiers? Modifiers
 function Menu:create_key_action(name, modifiers)
-	return self:create_action(function()
+	return function()
 		self.mouse_nav = false
 		self.modifiers = modifiers
 		self:maybe(name)
 		self.modifiers = nil
-	end)
+	end
 end
 
 function Menu:render()
@@ -691,11 +694,11 @@ function Menu:render()
 		end
 	end
 
-	cursor.on_primary_down = self:create_action(function() self:handle_cursor_down() end)
-	cursor.on_primary_up = self:create_action(function() self:handle_cursor_up() end)
+	cursor.on_primary_down = function() self:handle_cursor_down() end
+	cursor.on_primary_up = function() self:handle_cursor_up() end
 	if self.proximity_raw == 0 then
-		cursor.on_wheel_down = self:create_action(function() self:handle_wheel_down() end)
-		cursor.on_wheel_up = self:create_action(function() self:handle_wheel_up() end)
+		cursor.on_wheel_down = function() self:handle_wheel_down() end
+		cursor.on_wheel_up = function() self:handle_wheel_up() end
 	end
 
 	local ass = assdraw.ass_new()
@@ -724,7 +727,7 @@ function Menu:render()
 		ass:rect(menu_rect.ax, menu_rect.ay, menu_rect.bx, menu_rect.by, {color = bg, opacity = menu_opacity, radius = 4})
 
 		if is_parent and get_point_to_rectangle_proximity(cursor, menu_rect) == 0 then
-			cursor.on_primary_down = self:create_action(function() self:slide_in_menu(menu, x) end)
+			cursor.on_primary_down = function() self:slide_in_menu(menu, x) end
 		end
 
 		-- Draw submenu if selected
@@ -734,9 +737,7 @@ function Menu:render()
 			submenu_rect = draw_menu(current_item, menu_rect.bx + menu_gap, 1)
 			submenu_is_hovered = get_point_to_rectangle_proximity(cursor, submenu_rect) == 0
 			if submenu_is_hovered then
-				cursor.on_primary_down = self:create_action(function()
-					self:open_selected_item({preselect_first_item = false})
-				end)
+				cursor.on_primary_down = function() self:open_selected_item({preselect_first_item = false}) end
 			end
 		end
 

scripts/uosc/elements/Timeline.lua

@@ -17,6 +17,10 @@ function Timeline:init()
 	self.is_hovered = false
 	self.has_thumbnail = false
 
+	-- Delayed seeking timer
+	self.seek_timer = mp.add_timeout(0.05, function() self:set_from_cursor() end)
+	self.seek_timer:kill()
+
 	-- Release any dragging when file gets unloaded
 	mp.register_event('end-file', function() self.pressed = false end)
 end
@@ -109,6 +113,7 @@ function Timeline:on_prop_border() self:update_dimensions() end
 function Timeline:on_prop_fullormaxed() self:update_dimensions() end
 function Timeline:on_display() self:update_dimensions() end
 function Timeline:handle_cursor_up()
+	self.seek_timer:kill()
 	if self.pressed then
 		mp.set_property_native('pause', self.pressed.pause)
 		self.pressed = false
@@ -122,8 +127,10 @@ function Timeline:on_global_mouse_move()
 	if self.pressed then
 		self.pressed.distance = self.pressed.distance + get_point_to_point_proximity(self.pressed.last, cursor)
 		self.pressed.last.x, self.pressed.last.y = cursor.x, cursor.y
-		if state.is_video and math.abs(cursor.get_velocity().x) / self.width * state.duration > 30 then
+		if self.width / state.duration < 10 then
 			self:set_from_cursor(true)
+			self.seek_timer:kill()
+			self.seek_timer:resume()
 		else self:set_from_cursor() end
 	end
 end

scripts/uosc/intl/ru.json

@@ -1,59 +0,0 @@
-{
-    "Aspect ratio": "Соотношение сторон",
-    "Audio": "Аудио",
-    "Audio device": "Аудиоустройство",
-    "Audio devices": "Аудиоустройства",
-    "Audio tracks": "Аудиодорожки",
-    "Autoselect device": "Автовыбор устройства",
-    "Chapter %s": "Глава %s",
-    "Chapters": "Главы",
-    "Default": "По умолчанию",
-    "Default %s": "По умолчанию %s",
-    "Delete file & Next": "Удалить файл и след.",
-    "Delete file & Prev": "Удалить файл и пред.",
-    "Delete file & Quit": "Удалить файл и выйти",
-    "Disabled": "Отключено",
-    "Drives": "Диски",
-    "Edition": "Редакция",
-    "Edition %s": "Редакция %s",
-    "Editions": "Редакции",
-    "Empty": "Пусто",
-    "First": "Первый",
-    "Fullscreen": "Полный экран",
-    "Last": "Последний",
-    "Load": "Загрузить",
-    "Load audio": "Загрузить аудио",
-    "Load subtitles": "Загрузить субтитры",
-    "Load video": "Загрузить видео",
-    "Loop file": "Повторять файл",
-    "Loop playlist": "Повторять плейлист",
-    "Menu": "Меню",
-    "Navigation": "Навигация",
-    "Next": "Следующий",
-    "No file": "Нет файла",
-    "Open config folder": "Открыть папку конфигурации",
-    "Open file": "Открыть файл",
-    "Playlist": "Плейлист",
-    "Playlist/Files": "Плейлист / файлы",
-    "Prev": "Предыдущий",
-    "Previous": "Предыдущий",
-    "Quit": "Выйти",
-    "Screenshot": "Скриншот",
-    "Show in directory": "Показать в папке",
-    "Shuffle": "Перемешать",
-    "Stream quality": "Качество потока",
-    "Subtitles": "Субтитры",
-    "Track": "Дорожка",
-    "Track %s": "Дорожка %s",
-    "Utils": "Инструменты",
-    "Video": "Видео",
-    "%s channels": "%s канала/-ов",
-    "%s channel": "%s канал",
-    "default": "по умолчанию",
-    "drive": "диск",
-    "external": "внешняя",
-    "forced": "форсированная",
-    "open file": "открыть файл",
-    "parent dir": "родительская папка",
-    "playlist or file": "плейлист или файл"
-}

scripts/uosc/lib/std.lua

@@ -202,59 +202,3 @@ function Class:init() end
 function Class:destroy() end
 
 function class(parent) return setmetatable({}, {__index = parent or Class}) end
-
----@class CircularBuffer<T> : Class
-CircularBuffer = class()
-
-function CircularBuffer:new(max_size) return Class.new(self, max_size) --[[@as CircularBuffer]] end
-function CircularBuffer:init(max_size)
-	self.max_size = max_size
-	self.size = 0
-	self.pos = 0
-	self.data = {}
-end
-
-function CircularBuffer:insert(item)
-	self.pos = self.pos % self.max_size + 1
-	self.data[self.pos] = item
-	if self.size < self.max_size then self.size = self.size + 1 end
-end
-
-function CircularBuffer:get(i)
-	return i <= self.size and self.data[(self.pos + i - 1) % self.size + 1] or nil
-end
-
-local function iter(self, i)
-	if i == self.size then return nil end
-	i = i + 1
-	return i, self:get(i)
-end
-
-function CircularBuffer:iter()
-	return iter, self, 0
-end
-
-local function iter_rev(self, i)
-	if i == 1 then return nil end
-	i = i - 1
-	return i, self:get(i)
-end
-
-function CircularBuffer:iter_rev()
-	return iter_rev, self, self.size + 1
-end
-
-function CircularBuffer:head()
-	return self.data[self.pos]
-end
-
-function CircularBuffer:tail()
-	if self.size < 1 then return nil end
-	return self.data[self.pos % self.size + 1]
-end
-
-function CircularBuffer:clear()
-	for i = self.size, 1, -1 do self.data[i] = nil end
-	self.size = 0
-	self.pos = 0
-end

scripts/uosc/lib/utils.lua

@@ -1,26 +1,51 @@
 --[[ UI specific utilities that might or might not depend on its state or options ]]
 msg = require('mp.msg')
 
---- In place sorting of filenames
----@param filenames string[]
-function sort_filenames(filenames)
-	-- alphanum sorting for humans in Lua
-	-- http://notebook.kulchenko.com/algorithms/alphanumeric-natural-sorting-for-humans-in-lua
-	local function padnum(n, d)
-		return #d > 0 and ('%03d%s%.12f'):format(#n, n, tonumber(d) / (10 ^ #d))
-			or ('%03d%s'):format(#n, n)
+-- Sorting comparator close to (but not exactly) how file explorers sort files.
+sort_filenames = (function()
+	local symbol_order
+	local default_order
+
+	if state.platform == 'windows' then
+		symbol_order = {
+			['!'] = 1, ['#'] = 2, ['$'] = 3, ['%'] = 4, ['&'] = 5, ['('] = 6, [')'] = 6, [','] = 7,
+			['.'] = 8, ["'"] = 9, ['-'] = 10, [';'] = 11, ['@'] = 12, ['['] = 13, [']'] = 13, ['^'] = 14,
+			['_'] = 15, ['`'] = 16, ['{'] = 17, ['}'] = 17, ['~'] = 18, ['+'] = 19, ['='] = 20,
+		}
+		default_order = 21
+	else
+		symbol_order = {
+			['`'] = 1, ['^'] = 2, ['~'] = 3, ['='] = 4, ['_'] = 5, ['-'] = 6, [','] = 7, [';'] = 8,
+			['!'] = 9, ["'"] = 10, ['('] = 11, [')'] = 11, ['['] = 12, [']'] = 12, ['{'] = 13, ['}'] = 14,
+			['@'] = 15, ['$'] = 16, ['*'] = 17, ['&'] = 18, ['%'] = 19, ['+'] = 20, ['.'] = 22, ['#'] = 23,
+		}
+		default_order = 21
 	end
 
+	-- Alphanumeric sorting for humans in Lua
+	-- http://notebook.kulchenko.com/algorithms/alphanumeric-natural-sorting-for-humans-in-lua
+	local function pad_number(n, d)
+		return #d > 0 and ("%03d%s%.12f"):format(#n, n, tonumber(d) / (10 ^ #d))
+			or ("%03d%s"):format(#n, n)
+	end
+
+	--- In place sorting of filenames
+	---@param filenames string[]
+	return function(filenames)
 		local tuples = {}
-	for i, f in ipairs(filenames) do
-		tuples[i] = { f:lower():gsub('0*(%d+)%.?(%d*)', padnum), f }
+		for i, filename in ipairs(filenames) do
+			local first_char = filename:sub(1, 1)
+			local order = symbol_order[first_char] or default_order
+			local formatted = filename:lower():gsub('0*(%d+)%.?(%d*)', pad_number)
+			tuples[i] = {order, formatted, filename}
 		end
 		table.sort(tuples, function(a, b)
-		return a[1] == b[1] and #b[2] < #a[2] or a[1] < b[1]
+			if a[1] ~= b[1] then return a[1] < b[1] end
+			return a[2] == b[2] and #b[3] < #a[3] or a[2] < b[2]
 		end)
-	for i, tuple in ipairs(tuples) do filenames[i] = tuple[2] end
-	return filenames
+		for i, tuple in ipairs(tuples) do filenames[i] = tuple[3] end
 	end
+end)()
 
 -- Creates in-between frames to animate value from `from` to `to` numbers.
 ---@param from number
@@ -295,7 +320,7 @@ end
 -- Check if path is a protocol, such as `http://...`.
 ---@param path string
 function is_protocol(path)
-	return type(path) == 'string' and (path:find('^%a[%w.+-]-://') ~= nil or path:find('^%a[%w.+-]-:%?') ~= nil)
+	return type(path) == 'string' and (path:find('^%a[%a%d-_]+://') ~= nil or path:find('^%a[%a%d-_]+:\\?') ~= nil)
 end
 
 ---@param path string

scripts/uosc/main.lua

@@ -316,7 +316,8 @@ cursor = {
 	on_wheel_down = nil,
 	on_wheel_up = nil,
 	allow_dragging = false,
-	history = CircularBuffer:new(10),
+	history = {}, -- {x, y}[] history
+	history_size = 10,
 	-- Called at the beginning of each render
 	reset_handlers = function()
 		cursor.on_primary_down, cursor.on_primary_up = nil, nil
@@ -339,80 +340,9 @@ cursor = {
 			cursor.wheel_enabled = enable_wheel
 		end
 	end,
-	find_history_sample = function()
-		local time = mp.get_time()
-		for _, e in cursor.history:iter_rev() do
-			if time - e.time > 0.1 then
-				return e
-			end
-		end
-		return cursor.history:tail()
-	end,
-	get_velocity = function()
-		local snap = cursor.find_history_sample()
-		if snap then
-			local x, y, time = cursor.x - snap.x, cursor.y - snap.y, mp.get_time()
-			local time_diff = time - snap.time
-			if time_diff > 0.001 then
-				return { x = x / time_diff, y = y / time_diff }
-			end
-		end
-		return { x = 0, y = 0 }
-	end,
-	move = function(x, y)
-		local old_x, old_y = cursor.x, cursor.y
-
-		-- mpv reports initial mouse position on linux as (0, 0), which always
-		-- displays the top bar, so we hardcode cursor position as infinity until
-		-- we receive a first real mouse move event with coordinates other than 0,0.
-		if not state.first_real_mouse_move_received then
-			if x > 0 and y > 0 and x < INFINITY and y < INFINITY then state.first_real_mouse_move_received = true
-			else x, y = INFINITY, INFINITY end
-		end
-
-		-- Add 0.5 to be in the middle of the pixel
-		cursor.x, cursor.y = (x + 0.5) / display.scale_x, (y + 0.5) / display.scale_y
-
-		if old_x ~= cursor.x or old_y ~= cursor.y then
-			is_leave = cursor.x == INFINITY or cursor.y == INFINITY
-			is_enter = cursor.hidden and not is_leave;
-			if is_enter then
-				cursor.hidden = false
-				cursor.history:clear()
-				Elements:trigger('global_mouse_enter')
-			end
-			Elements:update_proximities()
-
-			if is_leave then
-				cursor.hidden = true
-				cursor.history:clear()
-
-				-- Slowly fadeout elements that are currently visible
-				for _, element_name in ipairs({'timeline', 'volume', 'top_bar'}) do
-					local element = Elements[element_name]
-					if element and element.proximity > 0 then
-						element:tween_property('forced_visibility', element:get_visibility(), 0, function()
-							element.forced_visibility = nil
-						end)
-					end
-				end
-
-				Elements:trigger('global_mouse_leave')
-			elseif not is_enter then
-				-- Update history
-				cursor.history:insert({x = cursor.x, y = cursor.y, time = mp.get_time()})
-			end
-
-			Elements:proximity_trigger('mouse_move')
-			cursor.queue_autohide()
-		end
-
-		render()
-	end,
-	leave = function () cursor.move(INFINITY, INFINITY) end,
 	-- Cursor auto-hiding after period of inactivity
 	autohide = function()
-		if not cursor.on_primary_up and not Menu:is_open() then cursor.leave() end
+		if not cursor.on_primary_up and not Menu:is_open() then handle_mouse_leave() end
 	end,
 	autohide_timer = (function()
 		local timer = mp.add_timeout(mp.get_property_native('cursor-autohide') / 1000, function() cursor.autohide() end)
@@ -425,12 +355,15 @@ cursor = {
 			cursor.autohide_timer:resume()
 		end
 	end,
-	-- Calculates distance in which cursor reaches rectangle if it continues moving on the same path.
+	-- Calculates distance in which cursor reaches rectangle if it continues moving in the same path.
 	-- Returns `nil` if cursor is not moving towards the rectangle.
 	direction_to_rectangle_distance = function(rect)
-		local prev = cursor.find_history_sample()
-		if not prev then return false end
-		local end_x, end_y = cursor.x + (cursor.x - prev.x) * 1e10, cursor.y + (cursor.y - prev.y) * 1e10
+		if cursor.hidden or not cursor.history[1] then
+			return false
+		end
+
+		local prev_x, prev_y = cursor.history[1][1], cursor.history[1][2]
+		local end_x, end_y = cursor.x + (cursor.x - prev_x) * 1e10, cursor.y + (cursor.y - prev_y) * 1e10
 		return get_ray_to_rectangle_distance(cursor.x, cursor.y, end_x, end_y, rect)
 	end
 }
@@ -531,7 +464,7 @@ function update_fullormaxed()
 	state.fullormaxed = state.fullscreen or state.maximized
 	update_display_dimensions()
 	Elements:trigger('prop_fullormaxed', state.fullormaxed)
-	cursor.leave()
+	update_cursor_position(INFINITY, INFINITY)
 end
 
 function update_human_times()
@@ -608,6 +541,61 @@ function set_state(name, value)
 	Elements:trigger('prop_' .. name, value)
 end
 
+function update_cursor_position(x, y)
+	local old_x, old_y = cursor.x, cursor.y
+
+	-- mpv reports initial mouse position on linux as (0, 0), which always
+	-- displays the top bar, so we hardcode cursor position as infinity until
+	-- we receive a first real mouse move event with coordinates other than 0,0.
+	if not state.first_real_mouse_move_received then
+		if x > 0 and y > 0 and x < INFINITY and y < INFINITY then state.first_real_mouse_move_received = true
+		else x, y = INFINITY, INFINITY end
+	end
+
+	-- Add 0.5 to be in the middle of the pixel
+	cursor.x, cursor.y = (x + 0.5) / display.scale_x, (y + 0.5) / display.scale_y
+
+	if old_x ~= cursor.x or old_y ~= cursor.y then
+		is_leave = cursor.x == INFINITY or cursor.y == INFINITY
+		is_enter = cursor.hidden and not is_leave;
+		if is_enter then
+			cursor.hidden, cursor.history = false, {}
+			Elements:trigger('global_mouse_enter')
+		end
+		Elements:update_proximities()
+
+		if is_leave then
+			cursor.hidden, cursor.history = true, {}
+			Elements:trigger('global_mouse_leave')
+		elseif not is_enter then
+			-- Update cursor history
+			for i = 1, cursor.history_size - 1, 1 do
+				cursor.history[i] = cursor.history[i + 1]
+			end
+			cursor.history[cursor.history_size] = {x, y}
+		end
+
+		Elements:proximity_trigger('mouse_move')
+		cursor.queue_autohide()
+	end
+
+	render()
+end
+
+function handle_mouse_leave()
+	-- Slowly fadeout elements that are currently visible
+	for _, element_name in ipairs({'timeline', 'volume', 'top_bar'}) do
+		local element = Elements[element_name]
+		if element and element.proximity > 0 then
+			element:tween_property('forced_visibility', element:get_visibility(), 0, function()
+				element.forced_visibility = nil
+			end)
+		end
+	end
+
+	update_cursor_position(INFINITY, INFINITY)
+end
+
 function handle_file_end()
 	local resume = false
 	if not state.loop_file then
@@ -685,9 +673,9 @@ function handle_mouse_pos(_, mouse)
 	if not mouse then return end
 	msg.trace("handle_mouse_pos: x:", mouse.x, ", y:", mouse.y, ", hover:", mouse.hover, ", hover_raw:", cursor.hover_raw, "first_real_received:", state.first_real_mouse_move_received)
 	if cursor.hover_raw and not mouse.hover then
-		cursor.leave()
+		handle_mouse_leave()
 	else
-		cursor.move(mouse.x, mouse.y)
+		update_cursor_position(mouse.x, mouse.y)
 	end
 	if state.first_real_mouse_move_received then
 		cursor.hover_raw = mouse.hover
@@ -991,7 +979,7 @@ bind_command('playlist', create_self_updating_menu_opener({
 	serializer = function(playlist)
 		local items = {}
 		for index, item in ipairs(playlist) do
-			local is_url = is_protocol(item.filename)
+			local is_url = item.filename:find('://')
 			local item_title = type(item.title) == 'string' and #item.title > 0 and item.title or false
 			items[index] = {
 				title = item_title or (is_url and item.filename or serialize_path(item.filename).basename),