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

Excel Tabs missing on bottom NC24 #313

Closed
AndyXheli opened this issue Jun 23, 2022 · 10 comments · Fixed by #370
Closed

Excel Tabs missing on bottom NC24 #313

AndyXheli opened this issue Jun 23, 2022 · 10 comments · Fixed by #370

Comments

@AndyXheli
Copy link

NC 24
image

NC 23
image

@AndyXheli AndyXheli changed the title Execl Tabs missing on bottom NC24 Excel Tabs missing on bottom NC24 Jun 23, 2022
@k-popov
Copy link

k-popov commented Jul 11, 2022

There is a workaround posted here: https://help.nextcloud.com/t/office-online-excel-tabs/140310/2 This helped in my case.

Install Custom CSS Plugin go: /settings/admin/theming In the settings, add a style that fixes the error:

#officeonline-wrapper
{
top: 0 !important;
}

@dario94
Copy link

dario94 commented Jul 18, 2022

There is a workaround posted here: https://help.nextcloud.com/t/office-online-excel-tabs/140310/2 This helped in my case.

Install Custom CSS Plugin go: /settings/admin/theming In the settings, add a style that fixes the error:

#officeonline-wrapper
{
top: 0 !important;
}

Hi, I can't find the path that you say. Could be /var/www/html/nextcloud/apps/theming/ ?
Here I have
[root@wp css]# cd /var/www/html/nextcloud/apps/theming/css
[root@wp css]# ls
settings-admin.scss theming.scss

@k-popov
Copy link

k-popov commented Jul 18, 2022

/settings/admin/theming is URL, not filesystem path. e.g. https://your-nextcloud-server/settings/admin/theming which should become available once you have the Custom CSS plugin

@AndyXheli
Copy link
Author

Thanks! @juliushaertl any feedback on this and what needs to be changed in the code. Maybe I can create a pull request with you help.

@dario94
Copy link

dario94 commented Jul 19, 2022

Thank you @k-popov !!!

@AndyXheli
Copy link
Author

hi @juliushaertl Do you have any updates on this for us looks like this app is not compatible with NC24 no just in this but also dose not lunch on the NC dashboard I think there might be a viewer issue

@ramiuslr
Copy link

There is a workaround posted here: https://help.nextcloud.com/t/office-online-excel-tabs/140310/2 This helped in my case.

Install Custom CSS Plugin go: /settings/admin/theming In the settings, add a style that fixes the error:

#officeonline-wrapper
{
top: 0 !important;
}

Helped a lot, thank you so much !

@centredecalcul
Copy link

Hello,
fix don't work in 25.
same problem in 25.0.7 with excel file opened through a shared link.
Can you re open this issue ?

@celevra
Copy link

celevra commented Nov 24, 2023

have that problem only on mobile, there are the tabs hidden.
I had no success with css

@overholted
Copy link

overholted commented Dec 30, 2023

I also can't see the tabs. I'm on 27.1.3. I tried the CSS above but it didn't work. I tried messing around with the CSS inspector in my browser and discovered that if I change the following top attribute from top: var(--header-height); to 20px the tabs become visible.

.viewer[data-v-9039c018] .modal-wrapper .modal-container {
top: 20px;
bottom: var(--header-height);
height: auto;
background-color: rgba(0,0,0,0);
box-shadow: none;
}

I don't think that's the right place to fix it, so I dug a little deeper and found that the problem appears to be the iframe that OnlyOffice is being pulled into. I tried adding some position and top attributes to the following element and it worked, but I think there may be a more elegant solution than position: relative:

.office-viewer__iframe[data-v-4e3ea991] {
width: 100%;
flex-grow: 1;
top: -27px;
position: relative;
}

So, for now, to fix this I added the following to my Theming>Custom CSS:

.office-viewer__iframe[data-v-4e3ea991] {
top: -27px!important;
position: relative!important;
}

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

Successfully merging a pull request may close this issue.

7 participants