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.

Sign up for GitHub

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

Jump to bottom

paragraph block divider line for <hr> not visible on safari #879

Closed
maxliqd opened this issue Oct 23, 2023 · 4 comments
Closed

paragraph block divider line for <hr> not visible on safari #879

maxliqd opened this issue Oct 23, 2023 · 4 comments
Assignees
@philli-m
Milestone
Handover Nov 23

Comments

@maxliqd
Copy link
Contributor

maxliqd commented Oct 23, 2023 •
edited
Loading

Essential Informationparagraph block divider line for html tag hr not visible on safari


URL: https://gd-dev.liqd.net/en/testbereich/test-detail-page/
device & browser: safari, mac
operating system (incl. version):
behaviour:
expected behaviour: should be visible like on chrome and firefox

Additional information if relevant
user role:
important screensize:
Comment/Question:

Screenshot if design related:
Since issues are public, make sure to not display personal or secret data.
@maxliqd maxliqd added this to the Handover Nov 23 milestone Oct 23, 2023
@philli-m philli-m self-assigned this Oct 26, 2023
philli-m added a commit that referenced this issue Oct 26, 2023
@philli-m
assets/base: restyle hr as BO styles break in safari fixes #879
3f219c3
philli-m added a commit that referenced this issue Oct 26, 2023
@philli-m
assets/base: restyle hr as BO styles break in safari fixes #879
5a607a4
philli-m added a commit that referenced this issue Oct 26, 2023
@philli-m
assets/base: restyle hr as BO styles break in safari fixes #879
b11c68c
@maxliqd
Copy link
Contributor Author

maxliqd commented Oct 27, 2023

Still does not work on dev in safari: https://gd-dev.liqd.net/en/testbereich/
Or is it not yet on dev?

@maxliqd maxliqd reopened this Oct 27, 2023
@philli-m
Copy link
Contributor

@maxliqd working locally so I think it might be the csp changes I added to test them as there's something off with dev, will let you know, sorry for confusion

@philli-m
Copy link
Contributor

@maxliqd sorry everything is working now, dev was on a csp testing branch, i'm so sorry! I didn't revert the change before leaving, I didn't know it didn't reset on a new build :(

@philli-m
Copy link
Contributor

working now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@philli-m philli-m

Labels
None yet
Projects
None yet
Milestone
Handover Nov 23
Development

No branches or pull requests
2 participants
@philli-m @maxliqd