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

Borders not respecting bordered=false parameter for static embedding #41393

Closed
air-rick opened this issue Apr 15, 2024 · 2 comments · Fixed by #41399 or #41432
Closed

Borders not respecting bordered=false parameter for static embedding #41393

air-rick opened this issue Apr 15, 2024 · 2 comments · Fixed by #41399 or #41432
Assignees
Labels
Embedding/ Use this label when unsure which flavor of embedding is impacted Priority:P1 Security holes w/o exploit, crashing, setup/upgrade, login, broken common features, correctness .Team/Embedding Type:Bug Product defects
Milestone

Comments

@air-rick
Copy link

Describe the bug

After the upgrade to Version 49, all of our embedded dashboards are showing with borders. We are using the following embed settings:

theme=transparent&bordered=false&titled=false

This seems to be a bug in Version 49 that was not present in version 48.

To Reproduce

  1. Create a dashboard
  2. Enable static embed
  3. Choose theme as transparent
  4. Choose bordered as false
  5. Choose titled as false
  6. Embed in website
  7. Refresh page, and dashboard will show up with borders

Expected behavior

There should not be any borders showing

Logs

No response

Information about your Metabase installation

{
  "browser-info": {
    "language": "en-US",
    "platform": "MacIntel",
    "userAgent": "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/123.0.0.0 Safari/537.36",
    "vendor": "Google Inc."
  },
  "system-info": {
    "file.encoding": "UTF-8",
    "java.runtime.name": "OpenJDK Runtime Environment",
    "java.runtime.version": "11.0.22+7",
    "java.vendor": "Eclipse Adoptium",
    "java.vendor.url": "https://adoptium.net/",
    "java.version": "11.0.22",
    "java.vm.name": "OpenJDK 64-Bit Server VM",
    "java.vm.version": "11.0.22+7",
    "os.name": "Linux",
    "os.version": "5.10.213-201.855.amzn2.x86_64",
    "user.language": "en",
    "user.timezone": "GMT"
  },
  "metabase-info": {
    "databases": [
      "snowflake",
      "postgres"
    ],
    "hosting-env": "unknown",
    "application-database": "postgres",
    "application-database-details": {
      "database": {
        "name": "PostgreSQL",
        "version": "14.10"
      },
      "jdbc-driver": {
        "name": "PostgreSQL JDBC Driver",
        "version": "42.7.2"
      }
    },
    "run-mode": "prod",
    "version": {
      "date": "2024-04-11",
      "tag": "v1.49.5",
      "hash": "31c4887"
    },
    "settings": {
      "report-timezone": null
    }
  }
}

Severity

Very annoying, affecting our branding in our product.

Additional context

No response

@air-rick air-rick added .Needs Triage Type:Bug Product defects labels Apr 15, 2024
@npretto npretto added Embedding/ Use this label when unsure which flavor of embedding is impacted .Team/Embedding Priority:P2 Average run of the mill bug and removed .Needs Triage labels Apr 15, 2024
@npretto
Copy link
Member

npretto commented Apr 15, 2024

@air-rick is the border only showing up on the top side?

@npretto npretto added Priority:P1 Security holes w/o exploit, crashing, setup/upgrade, login, broken common features, correctness and removed Priority:P2 Average run of the mill bug labels Apr 15, 2024
@npretto npretto self-assigned this Apr 15, 2024
@air-rick
Copy link
Author

air-rick commented Apr 15, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Embedding/ Use this label when unsure which flavor of embedding is impacted Priority:P1 Security holes w/o exploit, crashing, setup/upgrade, login, broken common features, correctness .Team/Embedding Type:Bug Product defects
Projects
None yet
2 participants