meissa
/
forgejo
7
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity
You cannot select more than 25 topics Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
forgejo-federated-star
forgejo
Branches Tags
${ item.name }
Create tag ${ searchTerm }
Create branch ${ searchTerm }
from '77f70bd5a3'
${ noResults }
forgejo/templates
History
wxiaoguang 1fcf96ad01
Improve PR Review Box UI (#22986) 
This PR follows: 
* #22950

### Before

The Review Box has many problems:

* It doesn't work for small screens.
* It has an anonying animation which makes the UI laggy.
* It uses "custom dropdown menu" which is very difficult to fine tune.
* `$().toggle('visible')` is not a correct call 
* jQuery just accepts any invalid `duration` argument:
`$().toggle('anyting')`
* The button is not a button.

<details>

![image](https://user-images.githubusercontent.com/2114189/219948865-6da3f39c-6fde-4c86-9e42-da5020f3d0c3.png)

</details>

### After

These problems are fixed, and eliminate many `!important` games.

<details>

![image](https://user-images.githubusercontent.com/2114189/219952744-8862fe1a-7ef1-49e4-bf92-2d0c1f104ee4.png)

![image](https://user-images.githubusercontent.com/2114189/219952771-be169a76-45fd-47a8-8f9c-b447d064f4ca.png)

![image](https://user-images.githubusercontent.com/2114189/219952784-3f52e9b7-64ce-4ad1-9553-64c33fb83042.png)

</details>

And most dropdown icons still looks good:

<details>

![image](https://user-images.githubusercontent.com/2114189/219952942-52866a00-e0f9-4af7-8fb5-eb1a8cad1ff3.png)

![image](https://user-images.githubusercontent.com/2114189/219948909-b3bfb844-f84e-4b79-ab1f-382ec66dec31.png)

</details>

Co-authored-by: delvh <leon@kske.dev>
 		1 year ago
..
admin Refactor hiding-methods, remove jQuery show/hide, remove `.hide` class, remove inline style=display:none (#22950) 1 year ago
api/packages/pypi Add Package Registry (#16510) 2 years ago
base Introduce customized HTML elements, fix incorrect AppUrl usages in templates (#22861) 1 year ago
code Remove Fomantic-UI's `.hidden` CSS class for menu elements (#22895) 1 year ago
custom Add footer extra links template (#9576) 4 years ago
explore Add context cache as a request level cache (#22294) 1 year ago
mail Use link in UI which returned a relative url but not html_url which contains an absolute url (#21986) 1 year ago
org Hide 2FA status from other members in organization members list (#22999) 1 year ago
package Introduce customized HTML elements, fix incorrect AppUrl usages in templates (#22861) 1 year ago
projects Scoped labels (#22585) 1 year ago
repo Improve PR Review Box UI (#22986) 1 year ago
shared Fix no user listed in org teams page (#22979) 1 year ago
status Move helpers to be prefixed with `gt-` (#22879) 1 year ago
swagger Add scopes to API to create token and display them (#22989) 1 year ago
user Add scopes to API to create token and display them (#22989) 1 year ago
home.tmpl Add main landmark to templates and adjust titles (#22670) 1 year ago
install.tmpl Refactor hiding-methods, remove jQuery show/hide, remove `.hide` class, remove inline style=display:none (#22950) 1 year ago
post-install.tmpl Introduce customized HTML elements, fix incorrect AppUrl usages in templates (#22861) 1 year ago