Skip to content

Adjust styling of language selection#317

Merged
Phu2 merged 1 commit into
mainfrom
301-lang-switch
Oct 2, 2025
Merged

Adjust styling of language selection#317
Phu2 merged 1 commit into
mainfrom
301-lang-switch

Conversation

@sroertgen

Copy link
Copy Markdown
Contributor

Adjustments as recommended in
#301 (comment)

And also made buttons round independent of used letters.

image

Deployed to dev for testing.

Adjustments as recommended in
#301 (comment)

And also made buttons round independent of used letters.
@sroertgen sroertgen requested a review from acka47 June 2, 2025 07:52
@sroertgen sroertgen added this to SkoHub Jun 2, 2025
@github-project-automation github-project-automation Bot moved this to Backlog in SkoHub Jun 2, 2025
@sroertgen sroertgen moved this from Backlog to Review in SkoHub Jun 2, 2025
@sroertgen sroertgen linked an issue Jun 2, 2025 that may be closed by this pull request
@acka47 acka47 removed a link to an issue Aug 5, 2025
@acka47 acka47 linked an issue Aug 5, 2025 that may be closed by this pull request
@acka47 acka47 removed this from SkoHub Aug 5, 2025
@acka47

acka47 commented Aug 15, 2025

Copy link
Copy Markdown
Member

Looks better than before (from https://test.skohub.io/acka47/testing-skohub-vocabs/heads/master/w3id.org/kim/hcrt/scheme.html):

image

It is strange, though, that the three language tags are not grouped evenly in threes over the two rows. If this could be adjusted easily, please do. Otherwise, +1.

@acka47 acka47 removed their assignment Aug 15, 2025

@acka47 acka47 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

See my other comment

@acka47 acka47 assigned Phu2 and unassigned sroertgen Sep 25, 2025
@acka47

acka47 commented Sep 25, 2025

Copy link
Copy Markdown
Member

As discussed today, we might as well merge as is as it is already an improvement. @Phu2 , let me know if you make adjustments and I need to do another review.

@Phu2

Phu2 commented Oct 2, 2025

Copy link
Copy Markdown
Contributor

@acka47 I think it's good enough. Please approve.

@acka47 acka47 self-requested a review October 2, 2025 07:31

@acka47 acka47 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

+1

@Phu2 Phu2 merged commit df63d58 into main Oct 2, 2025
3 checks passed
@Phu2 Phu2 deleted the 301-lang-switch branch October 2, 2025 07:38
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.

Adjust language switch UI

3 participants