-
Notifications
You must be signed in to change notification settings - Fork 399
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
META Gnoweb UI Improvements #3355
Comments
Add a way to easily see which network we are running on. This could be a ribbon or another indicator. We need distinct values for the portal, testnets, and local (gnodev). Bonus: subtle custom CSS allows us to have a gentle background color to quickly identify the portal, local, and other options. |
bug breaking CleanShot.2024-12-17.at.17.50.55.mp4 |
Lack of cookie for the key/address. CleanShot.2024-12-17.at.17.52.04.mp4 |
It would be nice to have dark mode |
This PR aims to fix some of remaining UI bugs on gnoweb after the revamp merge. Some of: #3355 - Fixes Safari select input design and icons - Fixes input hover - Fixes ToC font style - Fixes UI details and improve CSS - Fixes Responsive with long content - Fixes Scrollbar - Fixes fonts loading strategy and size - Fixes ts issue with copy btn (quick clicks) - Fixes some A11y
This PR aims to fix some of remaining UI bugs on gnoweb after the revamp merge. Some of: gnolang#3355 - Fixes Safari select input design and icons - Fixes input hover - Fixes ToC font style - Fixes UI details and improve CSS - Fixes Responsive with long content - Fixes Scrollbar - Fixes fonts loading strategy and size - Fixes ts issue with copy btn (quick clicks) - Fixes some A11y
Adds JavaScript features listed #3355 regarding Help section in gnoweb. - Adds localStorage feature for address - Fixes txlink - Minor UI styles
We should see if we can somehow make the page a little less empty when there is little or no content, like for example here: https://gno.land/r/docs/hello Also, in this specific case above, the TOC is not shown even though the text is H1 |
This PR aims to fix some of remaining UI bugs on gnoweb after the revamp merge. Some of: gnolang#3355 - Fixes Safari select input design and icons - Fixes input hover - Fixes ToC font style - Fixes UI details and improve CSS - Fixes Responsive with long content - Fixes Scrollbar - Fixes fonts loading strategy and size - Fixes ts issue with copy btn (quick clicks) - Fixes some A11y
Adds JavaScript features listed gnolang#3355 regarding Help section in gnoweb. - Adds localStorage feature for address - Fixes txlink - Minor UI styles
Naming auto-generated function calls for realms as 'docs' isn't very intuitive. It took me a few minutes to realize what it was. I initially thought it would lead to a general documentation page. |
Continue fixing the issues reported in #3355 mainly: - Colors a11y issues - Directory layout - Copy btn style - code element font-size to follow parent's one
This PR aims to fix some of remaining UI bugs on gnoweb after the revamp merge. Some of: #3355 - Fixes Safari select input design and icons - Fixes input hover - Fixes ToC font style - Fixes UI details and improve CSS - Fixes Responsive with long content - Fixes Scrollbar - Fixes fonts loading strategy and size - Fixes ts issue with copy btn (quick clicks) - Fixes some A11y
Adds JavaScript features listed #3355 regarding Help section in gnoweb. - Adds localStorage feature for address - Fixes txlink - Minor UI styles
Continue fixing the issues reported in #3355 mainly: - Colors a11y issues - Directory layout - Copy btn style - code element font-size to follow parent's one
if |
It seems that with a You can check on https://gno.land/r/docs/avl_pager |
Seems someone broke the TOC click to jump to anchor! Currently clicking just adds a single edit: fixed |
In the docs page, when user inputs for functions contain EDIT: solved |
This realm is a good example of reusing and integrating some of my recently added libraries: - `p/moul/md` #2819 - `p/moul/mdtable` #3100 - `p/moul/realmpath` #3257 - `p/moul/txlink` #3289 - `p/moul/collection` #3321 - `p/demo/avl/pager` #2584 It helped me identify that `txlink` was not escaping the arguments, which resulted in invalid links. (fixed in #3682) Additionally, it provided me with a better understanding of: - The shortcomings of the `p/moul/md` API, particularly regarding `"\n"` handling - The need for improved management of the pager for `p/moul/collection` - What kind of UI improvements we could need on gnoweb. #3355 Demo: https://github.com/user-attachments/assets/4b20cee8-b8d7-4eba-90a8-5b87a3c19521 I also suggest you to look at the `filetest.gno` file. I believe we should proceed with the merge, to inspire others to create similar composed realms. However, I have a few improvement ideas: 1. Extract most of the generic logic into a `p/moul/present`. 2. Consider either making r/moul/present importable by `r/coreteam/present` to create a hub for presentations from all teammates, or the opposite: make `r/coreteam/present` the content source and allow `r/moul/present` to display a subset where `author="moul"` What are your thoughts? 3. Clean up the code using an improved `p/moul/md` and possibly new `p/` generic utilities. Depenes on #3682 --------- Signed-off-by: moul <94029+moul@users.noreply.github.com>
This PR ensures that special characters like ! and ? in user inputs are properly escaped when generating commands in the docs page ($help). Previously, entering ! or any other special char could cause the command string to break by omitting a closing ", making it invalid. This fix applies proper escaping to prevent such issues, ensuring that generated commands remain valid and executable. The fix introduces an escaping function that handles shell-sensitive characters before inserting them into the generated command strings. This approach ensures the commands remain intact without affecting their output when executed. Thus, the escape char is also removed from the cmd when the shell-sensitive char is removed from the arg input. cf: [issue 3355](#3355 (comment))
This PR fixes an issue where gnoweb URLs contain both a query (? prefix) and a gnowebquery ($ prefix). Previously, the header links appended the gnowebquery after the query, whereas it should be placed before the query. This update ensures the correct ordering of URL components. cf issue: #3355 (comment) --------- Co-authored-by: Guilhem Fanton <8671905+gfanton@users.noreply.github.com>
Let's add copy buttons and codelines to all rendered codeblocks. I see now downside to this. Any thoughts? |
Yeah I have an issue for that. I plan to tackle this meta data PR next week!👍 |
Centralizing here some feedback about the new Gnoweb UI.
The text was updated successfully, but these errors were encountered: