Styleguide bugs on 'Buttons' page

The image presents a screenshot of a style guide for atoms, showcasing a list of buttons with their corresponding descriptions and examples. (Captioned by AI)

I am new to the styleguide (and discourse). The styleguide appears to have a number of bugs, such as once you go to the button styleguide you cannot leave unless you refresh the page. I had to delete my header for privacy reasons else I would have shown the refresh loading the appropriate page.

Other things:

Thanks!

4 Likes

This one is intentional behaviour across the whole of Discourse, so that our PWA matches the behaviour of normal websites. If you click a link to a different page, you will be scrolled to the top of the new page.

Do you have a particularly large header on your site? If so, I can see why it would be frustrating :thinking:

Thanks for reporting! Will be fixed by this PR

1 Like

My own styleguide works fine, though on first load, the styleguide here on meta did freeze up. After a refresh it all worked properly. Might be an issue with some heavy script timing out. Depending on the browser, I sometimes see random Discourse pages time out before loading completely.

(& I see david has addressed the missing button examples.)

Thanks for the fix.

My expectation for the sidebar has been for the page not to return to the top, similar to mdn:

Header isnā€™t crazy large by any means, I just found it bothersome when trying to learn the inā€™s and outā€™s of the styleguide and was clicking my way down the sidebar that I had to scroll back down to the correct sidebar location after every page load.

1 Like

Interesting, when you navigate to your btn styleguide are you able to switch to a different styleguide page by clicking the sidebar WITHOUT refreshing the page? I am completely locked in on local dev and in production.

1 Like

That will be fixed by my ā€˜flat-buttonā€™ PR above. Itā€™s now merged, so if you pull the latest main, things should be much better.

Unfortunately after a rendering error like that, Ember gets into a bad state, which causes other seemingly-unrelated things to break.

3 Likes

If you wanna squash another bug lolā€¦ I tried to change my emoji reaction from a :+1: to :rocket:

Looks like thereā€™s a TS error preventing the change (sorry lol)

1 Like

weird now itā€™s working for meā€¦ ĀÆ\_(惄)_/ĀÆ

1 Like

This topic was automatically closed 3 days after the last reply. New replies are no longer allowed.