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

XWIKI-21631: Tour generates a bottom scroll-bar, when items are created near the margins #3616

Merged
merged 1 commit into from
Oct 31, 2024

Conversation

Sereza7
Copy link
Contributor

@Sereza7 Sereza7 commented Oct 31, 2024

Jira URL

https://jira.xwiki.org/browse/XWIKI-21631

Changes

Description

  • Removed a useless class from the popover-navigation
  • Overwrote a style from bootstrap-tour with a precise selector

Clarifications

  • The combinaison of negative margin with positive padding was the cause of the bug.
  • I removed the .row class from bootstrap that brought the negative margin.
  • I overwrote the .popover-navigation class from bootstrap-tour that brought the padding.
  • The selector I used to overwrite is very specific, this new style rules should not collide and break any other components or customization.

Screenshots & Video

Before the PR, it looked the same, but on Firefox 128, I could see a horizontal scrollbar trying to scroll would twitch with a blank line on the right and end up as non scrolled.
After the changes proposed in this PR were applied on my local instance, here are the results:
Screenshot from 2024-10-31 11-06-51
There is no more scrollbar. The style of the box did not change.

Executed Tests

Only manual tests, see above.
From what I can see, the pageObject used in tests is not broken:

getDriver().findElement(By.xpath("//div[contains(@class, 'popover-navigation')]//a[@id='bootstrap_tour_end']"))
.click();

The xpath selectors do not rely on the "style only" .row class but only on .popover-navigation, which is kept the same.
I did not run any automated tests.

Expected merging strategy

  • Prefers squash: Yes
  • Backport on branches:
    • 16.4.X
    • 15.10.X

…ed near the margins

* Removed a useless class from the popover-navigation
* Overwrote a style from bootstrap-tour with a precise selector
@Sereza7 Sereza7 added backport stable-15.10.x Used for automatic backport to 15.10.x branch. backport stable-16.4.x and removed backport stable-15.10.x Used for automatic backport to 15.10.x branch. labels Oct 31, 2024
@acotiuga acotiuga merged commit ae4f4e1 into xwiki:master Oct 31, 2024
3 checks passed
github-actions bot pushed a commit that referenced this pull request Oct 31, 2024
…ed near the margins (#3616)

* Removed a useless class from the popover-navigation
* Overwrote a style from bootstrap-tour with a precise selector

(cherry picked from commit ae4f4e1)
surli pushed a commit that referenced this pull request Nov 4, 2024
…ed near the margins (#3616)

* Removed a useless class from the popover-navigation
* Overwrote a style from bootstrap-tour with a precise selector

(cherry picked from commit ae4f4e1)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants