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

Tab graphic bug #912

Closed
gmikhail opened this issue Dec 26, 2022 · 13 comments · Fixed by #1046
Closed

Tab graphic bug #912

gmikhail opened this issue Dec 26, 2022 · 13 comments · Fixed by #1046
Assignees
Milestone

Comments

@gmikhail
Copy link

Unnecessary tab separator is displayed when opening a new tab:

And when you close tab it disappears:

I also think that opening a blank tab with no content is itself a bug. At least I did not find how to interact with it (search doesn't work on it).

Version: 2.3.1
Windows 10 Pro 22H2 (build 19045.2364)

@kelson42 kelson42 added the bug label Dec 26, 2022
@kelson42 kelson42 modified the milestones: 2.3.1, 2.4.0 Dec 26, 2022
@kelson42
Copy link
Collaborator

@gmikhail If you run kiwix-serve.exe —version, what is printed?

@gmikhail
Copy link
Author

@kelson42 I don't have this file. kiwix-desktop.exe does not find this command. The installer was downloaded from the official website: https://download.kiwix.org/release/kiwix-desktop/kiwix-desktop_windows_x64.zip (redirect to https://mirrors.dotsrc.org/kiwix/release/kiwix-desktop/kiwix-desktop_windows_x64_2.3.1.zip)

@kelson42
Copy link
Collaborator

@gmikhail Ok, sorry might be kiwix.exe, but will check myself.

@kelson42
Copy link
Collaborator

@juuz0 Are you able to reproduce this issue?

@goutnet
Copy link

goutnet commented May 14, 2023

Hi @kelson42 I gave it a try on last main (but can't repro):

image

@gmikhail could you maybe provide more specific instructions to reproduce this issue?

@gmikhail
Copy link
Author

Hi @goutnet

Latest version from the official website still has this problem (download link redirect to https://ftp.fau.de/kiwix/release/kiwix-desktop/kiwix-desktop_windows_x64_2.3.1-2.zip).

Judging by window appearance, you are using Linux, while I am using the Windows version. Maybe it's a Windows-specific Qt thing? Because arrows for switching tabs on overflow is also different.

GIF

Unfortunately, I do not currently have the opportunity to compile Kiwix from source to check if there is a bug.

Also in your screenshot you can see that you can open several empty tabs. Which in itself is also wrong behavior because an empty tab cannot be interacted with (I mentioned about this in the first message).

@goutnet
Copy link

goutnet commented May 14, 2023

correct, I am using Linux indeed. I don't have a Windows env running just now (will set it up later).

Regarding the empty tabs though, that one I confirm I can see it on Linux. @kelson42 is this a bug or a feature? (aka: should empty tabs be allowed?), let me know :)

@kelson42
Copy link
Collaborator

kelson42 commented May 15, 2023

Empty tab clicking on + is « normal ». Feature is not fully implemented (it’s a duplicate of #499).

The problem is only the visible surnumerous vertical bar. If not buggy by us, this sounds like an upstream bug which should be reported/identified.

@kelson42
Copy link
Collaborator

The problem is only the visible surnumerous vertical bar. If not buggy by us, this sounds like an upstream bug which should be reported/identified.

I strongly suspect here a bug with Qt 5.13 used in the built, here (on my dev system) with Qt 5.15 I don't have the problem. We will wait to update the version Qt to see if we still have this problem. It's not important enough IMO to invest more time for the moment.

@kelson42 kelson42 self-assigned this Aug 11, 2023
@kelson42 kelson42 modified the milestones: 2.4.0, 2.5.0 Aug 11, 2023
@kelson42 kelson42 modified the milestones: 2.5.0, 2.4.0 Mar 1, 2024
@kelson42 kelson42 mentioned this issue Mar 1, 2024
5 tasks
@kelson42 kelson42 assigned kelson42 and unassigned kelson42 Mar 4, 2024
@juuz0
Copy link
Collaborator

juuz0 commented Sep 9, 2024

@kelson42 imo this still exists (and also unnecessary thickness on top of tabs) - on latest app image.
image

Pretty simple fix though, just make border-right and border-top 0 for tabbar (but let me know if you can reproduce)

@kelson42
Copy link
Collaborator

kelson42 commented Sep 9, 2024

@juuz0 what is the version of Qt you use?

@juuz0
Copy link
Collaborator

juuz0 commented Sep 9, 2024

@kelson42 5.15 but should it matter on an appimage?

@kelson42
Copy link
Collaborator

kelson42 commented Sep 9, 2024

I have 5.15.13 and it works fine for me. I don't know for appimage, but I'm sure there is nothing we can do anyway and that the bug is probably fixed in most recent version of Qt.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants