Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • B bootstrap
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 263
    • Issues 263
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 114
    • Merge requests 114
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Bootstrap
  • bootstrap
  • Issues
  • #14607
Closed
Open
Issue created Sep 12, 2014 by Administrator@rootContributor

Popover would benefit from class indicating whether title was hidden.

Created by: kevinparkerson

While the popover control is really nice, the dynamic nature of the displayed markup makes it difficult to apply styling for themes. The title in particular is a problem I'm running into - it is set to "display: hidden" via javascript if not declared in the options upon initialization. Since there are no on-page indicators that this has happened, it is difficult to style the arrow appropriately. Here is a demonstration of the problem on my public dropbox.

What I'm proposing is that the popover control add a class such as no-title to the parent container of the dynamically generated markup to indicate that the title has been hidden. This way themes can have more control over the popover arrow colors in different scenarios. I'd actually recommend that the class itself also be responsible for hiding the title vs an inline style applied via javascript, but since that code has already been written that way it might be too jarring to developers to change now.

I will make a pull request for this issue asap.

Assignee
Assign to
Time tracking