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

Annotate uu-internal links #316

Open
wants to merge 15 commits into
base: staging
Choose a base branch
from
Open

Annotate uu-internal links #316

wants to merge 15 commits into from

Conversation

sebix
Copy link
Member

@sebix sebix commented May 5, 2019

Based on #205 and adds annotations for wiki, post, topic, forum and github.

Copy link
Member

@chris34 chris34 left a comment

Choose a reason for hiding this comment

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

Just wanted to cherry-pick the two commits into the interwikilink PR, but then i stumbled upon some things…

padding-left: 18px;
}
a[href^="https://forum.ubuntuusers.de/post/"] {
background-image: url(../img/icon_minipost.gif);
Copy link
Member

Choose a reason for hiding this comment

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

IMHO this will display the minipost-gif two times beneath a forum post, as there the current minipost-icon is includeded via <img> https://github.com/inyokaproject/theme-ubuntuusers/blob/staging/inyoka_theme_ubuntuusers/templates/forum/topic.html#L288

I dont know if :not(.postinfo) a[href^=…] or something like this could help.

(For interwikilinks this should be working, as they also use a background-image. Nevertheless, it could confuse users, if there exists also an interwikilink for the same url, but with a different icon.)

Copy link
Member Author

Choose a reason for hiding this comment

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

IMHO this will display the minipost-gif two times beneath a forum post, as there the current minipost-icon is includeded via <img> https://github.com/inyokaproject/theme-ubuntuusers/blob/staging/inyoka_theme_ubuntuusers/templates/forum/topic.html#L288

True

I dont know if :not(.postinfo) a[href^=…] or something like this could help.

AFAIK (with :not) you can't select (an attribute of) a parent. We could just remove the img and permalink the full date (which is better usability IMO)

(For interwikilinks this should be working, as they also use a background-image. Nevertheless, it could confuse users, if there exists also an interwikilink for the same url, but with a different icon.)

@@ -38,6 +38,27 @@ a.team {
a.search {
background-image: url(../img/interwiki/search.png);
}
a[href^="https://wiki.ubuntuusers.de/"] {
Copy link
Member

Choose a reason for hiding this comment

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

I question, if hardcoding domains is good.

Copy link
Member Author

Choose a reason for hiding this comment

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

In this case the line is kind of a fallback.

In case the user uses the [:wiki:] syntax, the usual css styling applies - or can we apply (we currently don't), with no hardcoded domain.
In case the user uses the generic link syntax (and "hardcodes" the domain), we can still apply the styling by hardcoding the domain too.

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