THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY.

A new Flyspray task has been opened.  Details are below. 

User who did this - Wei Peng (pengw) 

Attached to Project - awesome
Summary - dynamic tagging regression with commit 
9c69e857edb6690fe5a496e82ec57a0bbae36702
Task Type - Bug Report
Category - awful
Status - Unconfirmed
Assigned To - 
Operating System - Linux
Severity - High
Priority - Normal
Reported Version - git/master
Due in Version - Undecided
Due Date - Undecided
Details - commit 9c69e857edb6690fe5a496e82ec57a0bbae36702 (first in v3.5.3) 
introduce the following change to the function "awful.tag.setscreen" 
lib/awful/tag.lua.in, which breaks "awful.tag.move" in the same file.

tag.setproperty(t, "index", nil)

In "awful.tag.move,"

        tag.setproperty(tmp_tag, "index", i)
        tag.setscreen(tmp_tag, scr)

is used to assign index to the shuffled tags. However, the change to 
awful.tag.setscreen will obliterate the newly assigned order.

This will introduce problem in dynamic tagging (e.g., 
https://bbs.archlinux.org/viewtopic.php?id=179477 and 
https://github.com/pw4ever/awesome-wm-config/blob/master/rc.lua#L529), which 
the taglist will sort the tag based on its "index."

Recommendation: revert the change "tag.setproperty(t, "index", nil)" change 
mentioned above.

More information can be found at the following URL:
https://awesome.naquadah.org/bugs/index.php?do=details&task_id=1249

You are receiving this message because you have requested it from the Flyspray 
bugtracking system.  If you did not expect this message or don't want to 
receive mails in future, you can change your notification settings at the URL 
shown above.

--
To unsubscribe, send mail to awesome-devel-unsubscr...@naquadah.org.

Reply via email to