Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Show tab-line when tab-line-mode is enabled #899

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

mitch-kyle
Copy link

No description provided.

…ctional as is in floating windows so hide the tab-line unless explicitly set in configuration
@sarg
Copy link

sarg commented Mar 31, 2023

Confirming that it works for me fine.
@medranocalvo have you tried it? Seems like a good addition, as exwm supports tab-bar-mode, mode-line, but not tab-line.

@medranocalvo
Copy link
Collaborator

Hello @mitch-kyle.

Let's start with the apologies: sorry for the long wait.

I had a look at the change and it looks good. I think I'd prefer both commits to be merged, but I can take care of that and the commit message if that's OK for you.

I couldn't find your copyright assignment. EXWM is part of GNU Emacs; as a consequence, contributions must have their copyright assigned to the FSF. Have you already assigned copyright? Otherwise, are you familiar with this matter?

@sarg, thank you for testing this change.

@mitch-kyle
Copy link
Author

Oh no problem, do what you have to do. 👍

I've heard of the assignment process, I can see about getting that done in the next few days

@medranocalvo
Copy link
Collaborator

Wonderful, thank you. Have a look at https://www.gnu.org/software/emacs/manual/html_node/emacs/Copyright-Assignment.html for info regarding copyright assignment.

@walseb
Copy link
Contributor

walseb commented Dec 20, 2023

Thanks for your work on this issue!

Any update on this? I ran into this problem today.

@walseb
Copy link
Contributor

walseb commented Jan 5, 2024

@mitch-kyle ping :)

@mitch-kyle
Copy link
Author

I don't have a working printer or scanner to fulfill gnu's archaic business processes and it's not worth it to me at this point. I wrote this freely for this project. Use it or don't.

@mitch-kyle mitch-kyle closed this Jan 6, 2024
@mitch-kyle mitch-kyle reopened this Jan 10, 2024
@mitch-kyle
Copy link
Author

Reopened to be merged whenever appropriate

@minad
Copy link
Contributor

minad commented Jan 11, 2024

@mitch-kyle This has already been discussed and sorry for bringing it up again - do you have the chance and are you willing to sign the FSF copyright papers? In my case, the whole process was rather smooth and went completely via email. I don't recall, either I signed via a wacom display or I copied my signature as an image into the form. The whole process didn't take too long. But I understand that it is demotivating that this has to be done before your change can be merged.

@medranocalvo
Copy link
Collaborator

@minad, @mitch-kyle has assigned copyright, we have to wait for confirmation. Sorry, I should have written this earlier.

I think we should release 0.29 with current changes and after that merge XSETTINGS and tab-line, and leave both some time on master. But first let's move to the org so that I don't block anymore.

@mitch-kyle: thank you very much, double thanks for your work and your willingness to help.

@minad
Copy link
Contributor

minad commented Jan 12, 2024

@mitch-kyle @medranocalvo Thank you both! @Stebalien and I will handle this PR after the migration.

@minad
Copy link
Contributor

minad commented Jan 13, 2024

@medranocalvo @Stebalien I think a 0.29 release is not needed right now given the small number of changes since 0.28. I prefer if we don't block additions like xsettings if they have to be enabled explicitly anyway. I haven't tried/checked this tab-line patch yet, but it might also be simple enough to get in now.

@medranocalvo
Copy link
Collaborator

@medranocalvo @Stebalien I think a 0.29 release is not needed right now given the small number of changes since 0.28. I prefer if we don't block additions like xsettings if they have to be enabled explicitly anyway. I haven't tried/checked this tab-line patch yet, but it might also be simple enough to get in now.

Oh, that's definitely alright. We can merge @mitch-kyle's patch as well, but then shouldn't release until we have confirmation that the papers got through. I've asked Stefan Monnier to check, will post once we have confirmation.

@medranocalvo
Copy link
Collaborator

Not yet filed. I'll ask again in a week or two.

@medranocalvo
Copy link
Collaborator

Not yet filed. I'll ask again in a week or two.

Not yet on file. I'll ask again in a week or two.

@progfolio
Copy link

Surely it must be on file by now.

@medranocalvo
Copy link
Collaborator

I asked again on 31 March with no answer.

I have now asked again @mitch-kyle and the GNU ELPA Maintainer.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants