Skip to content
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

Unable to make a tabbed layout #3002

Closed
ghost opened this issue Oct 27, 2018 · 9 comments
Closed

Unable to make a tabbed layout #3002

ghost opened this issue Oct 27, 2018 · 9 comments
Labels
bug Not working as intended

Comments

@ghost
Copy link

ghost commented Oct 27, 2018

To reproduce:
Go to a new workspace
Create a new terminal
Set its layout to vertical
Try to set its layout to tabbed
Nothing will happen.

What this prevents:
Having 2 containers, side by side, with one being tabbed.

@emersion emersion added the bug Not working as intended label Oct 27, 2018
@RyanDwyer
Copy link
Member

Works fine for me. When I set the layout to vertical I get the indicator along the bottom. Then when I set the layout to tabbed the indicator disappears. Note that the view still retains its border styling while there's only one child in the tabbed container, so if you're using border pixel then it'll still have a pixel border. Then when I open a second terminal, the tabbed title bar appears.

What this prevents:
Having 2 containers, side by side, with one being tabbed

Your instructions don't produce this layout.

@ghost
Copy link
Author

ghost commented Oct 28, 2018

They don't because I wanted the simplest possible version to replicate.
I'm using no borders at all, that's maybe related?

@RyanDwyer
Copy link
Member

Works with default_border none as well. Can you please attach a debug log?

@progandy
Copy link
Contributor

Try to set its layout to tabbed
Nothing will happen.

I'm using no borders at all, that's maybe related?

When there are no borders, then the master branch doesn't show the tabs if there is only one window, you have to open at least two. (65328ef)

@ghost
Copy link
Author

ghost commented Oct 28, 2018

I think that's wrong, I preferred the default behavior - to open one terminal, make it tabbed, and then open new tabs as I need.
I'd agree to make it a setting, but forcing this on users isn't nice.

@ddevault
Copy link
Contributor

I think that's wrong, I preferred the default behavior - to open one terminal, make it tabbed, and then open new tabs as I need.

I agree, we should probably do this.

@johnae
Copy link
Contributor

johnae commented Oct 30, 2018

Afaik the new behavior is how i3 works which is why it was changed in sway. Doesn't mean it's the best way to do it ofc.

@ddevault
Copy link
Contributor

Can anyone start up an i3 session to confirm that claim?

@progandy
Copy link
Contributor

@SirCmpwn I still run i3 most of the time, here are some tests:

$ i3 --version
i3 version 4.15 (2018-03-10) © 2009 Michael Stapelberg and contributors
$ cat /tmp/i3.conf 
hide_edge_borders none
default_border pixel

set $mod Mod4

bindsym $mod+v split v
bindsym $mod+b split h
bindsym $mod+t layout toggle splitv splith tabbed stacked
bindsym $mod+s layout stacked

bindsym $mod+Shift+c reload
bindsym $mod+Shift+r restart
bindsym $mod+Shift+e exec --no-startup-id i3-msg exit
bindsym $mod+Return exec termite

# change focus
bindsym $mod+h focus left
bindsym $mod+j focus down
bindsym $mod+k focus up
bindsym $mod+l focus right

# move focused window
bindsym $mod+Shift+h move left
bindsym $mod+Shift+j move down
bindsym $mod+Shift+k move up
bindsym $mod+Shift+l move right

hide_edge_borders smart, default_border pixel

  • 2 terminals open
    splitv, splith - no titlebar
    tabbed, stacked - titlebar visible
  • 1 terminal open
    splitv, splitv - no titlebar
    tabbed, stacked - no titlebar

hide_edge_borders smart, default_border normal

  • titlebars always visible

hide_edge_borders none, default_border pixel

  • 2 terminals open
    splitv, splith - no titlebar
    tabbed, stacked - titlebar visible
  • 1 terminal open
    splitv, splitv - no titlebar
    tabbed, stacked - no titlebar
  • 2 terminals side by side (splith), left terminal in a tabbed container:
    • no titlebar visible
    • while focusing on the left terminal, open another one
    • left tabbed container shows two terminals with tabbar.

thejan2009 added a commit to thejan2009/sway that referenced this issue Feb 24, 2019
Behaves same as _both_ in i3wm with regards to tabbed/stacked containers. If
there is only one tabbed/stacked container child, title bar is hidden.

Related issues and merge requests: swaywm#3031, swaywm#3002, swaywm#2912, swaywm#2987.
thejan2009 added a commit to thejan2009/sway that referenced this issue Feb 24, 2019
Behaves same as _both_ in i3wm with regards to tabbed/stacked containers. If
there is only one tabbed/stacked container child, title bar is hidden.

Related issues and merge requests: swaywm#3031, swaywm#3002, swaywm#2912, swaywm#2987.
thejan2009 added a commit to thejan2009/sway that referenced this issue Feb 24, 2019
Behaves same as _both_ in i3wm with regards to tabbed/stacked containers. If
there is only one tabbed/stacked container child, title bar is hidden.

Related issues and merge requests: swaywm#3031, swaywm#3002, swaywm#2912, swaywm#2987.
thejan2009 added a commit to thejan2009/sway that referenced this issue Feb 24, 2019
Enables i3-compatible behavior regarding hiding the title bar on tabbed and
stacked containers with one child.

Related issues and merge requests: swaywm#3031, swaywm#3002, swaywm#2912, swaywm#2987.
thejan2009 added a commit to thejan2009/sway that referenced this issue Feb 24, 2019
Enables i3-compatible behavior regarding hiding the title bar on tabbed and
stacked containers with one child.

Related issues and merge requests: swaywm#3031, swaywm#3002, swaywm#2912, swaywm#2987.
RedSoxFan pushed a commit that referenced this issue Feb 24, 2019
Enables i3-compatible behavior regarding hiding the title bar on tabbed and
stacked containers with one child.

Related issues and merge requests: #3031, #3002, #2912, #2987.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Not working as intended
Development

No branches or pull requests

5 participants