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

Rules text and styling preview #558

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

FredrikBergelin
Copy link

@FredrikBergelin FredrikBergelin commented Oct 25, 2024

Updated rules text and styling for all markdown pages. More changes will follow, team want to review the current changes on staging environment.

Also snowflakes are set to start in December, not October.

// FreadyFish

@Sheikah45
Copy link
Member

The only thing I am curious about is why the paths need to be hidden instead of just using the test server. @Brutus5000

@FredrikBergelin
Copy link
Author

No reason, just me not doing my homework before I start macgyvering. Will fix

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Was not written with a Russian R (the one that looks like "P")

@Brutus5000
Copy link
Member

The only thing I am curious about is why the paths need to be hidden instead of just using the test server. @Brutus5000
It has to be considered, that these kind of public markdown files are used in other places (e.g. a forum post or github which both also accept markdown). In these cases we need an absolute url.

@FredrikBergelin Can you add a screenshot of what even has changed?

@FredrikBergelin
Copy link
Author

Capture

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Even though this became unused after reverting, it might prove useful so I left it in. It allows us to inject a class to the container so each page can be styled individually.

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 this pull request may close these issues.

3 participants