-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
OF-2892 Available Plugins page: redesign to make Documentation easier to see #2541
Conversation
3b217ed
to
5917a86
Compare
xmppserver/src/main/java/org/jivesoftware/openfire/update/AvailablePlugin.java
Show resolved
Hide resolved
I've not tested this (yet) but this looks to be an awesome change! |
5917a86
to
dfe92df
Compare
Pushed the rebase. |
…ame instead of plugin.available.open_source And remove the unused open_source key
dfe92df
to
1e632dc
Compare
I fixed the bug with an additional column in the 1e632dc Also I rebased on top of main branch. So the PR can be merged |
Thanks! I have added a small change to this: I always have trouble tracking the lines of each plugin. To have a bit of a better visual indication, I made the background of a plugin-line change color slightly (from light-grey to white) when the mouse cursor hovers over it. |
I don't think that that the failure of the Docker image build is related to changes in this PR. I'm happy to ignore it. |
The Plugins and Available plugins pages have two icons for documentation (README) and changelog.
The icons are very small and placed together that makes it difficult to understand them.
The icons are only 16px and the only way to understand is to put a cursor over them to see a title. This wouldn't be possible to to from a mobile at all (minor issue).
As a solution we can switch to a table where each cell can have a few lines.
I moved the changelog icon into the Version column as a an additional row. The re
Similarly the Documentation link moved into the Description column, and the Description merged into the Plugin Name.