bugGNU Screen - Bugs: bug #29800, layout next and prev reversed?

 
 

bug #29800: layout next and prev reversed?

Submitter:  Curtis Brown <mrbrown8>
Submitted:  Thu 06 May 2010 04:27:23 AM UTC
   
 
Category:  Program Logic Severity:  3 - Normal
Priority:  * 5 - Normal Status:  Fixed
Privacy:  Public Assigned to:  None
Open/Closed:  Closed Release:  Cur Dev Sources
Fixed Release:  4.1.0 Planned Release:  4.1.0
Work Required:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Tue 22 Jun 2010 03:24:06 PM UTC, comment #2: 

The 'more correct' fix is to make sure the layouts are ordered correctly in the list. Fixed in a805439:

     Layouts were being stored in a reversed list. This makes it
     confusing for both users and developers with regards to what
     the next layout is ('next', lay_next). So store the layouts
     in a properly ordered list in order to avoid the confusion.

Thanks for reporting the bug!

Sadrul Habib Chowdhury <sadrul>
Group administrator
Sat 08 May 2010 03:50:49 PM UTC, comment #1: 


Attaching a simple patch that would fix this. TIA

(file #20481)

Curtis Brown <mrbrown8>
Thu 06 May 2010 04:27:23 AM UTC, original submission:  

When I have more than one layout in a session, using layout next and layout prev seem backwards.

In a session, I have:

:layout show

  1. layout0  1* layout1  2 layout2


But using next, I get:

:layout next
:layout show
0* layout0  1 layout1  2 layout2

:layout next
:layout show

  1. layout0  1 layout1  2* layout2


:layout next
:layout show

  1. layout0  1* layout1  2 layout2


This is similar to prev as well:

:layout prev
:layout show

  1. layout0  1 layout1  2* layout2


Perhaps I'm not doing something right?

Curtis Brown <mrbrown8>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

Attached Files

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by sadrul (Posted a comment)
  • -email is unavailable- added by mrbrown8 (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follow 4 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2010-06-22 sadrul StatusNone Fixed
        Open/ClosedOpen Closed
        Fixed ReleaseNone 4.1.0
    2010-05-08 mrbrown8 Attached File- Added patch_bug29800_layout_next_and_prev.diff, #20481

    Back to the top

    Powered by Savane 3.13-bb6a.
    Corresponding source code