Skip to content
This repository was archived by the owner on Sep 8, 2021. It is now read-only.

Renamed more link to refresh (what it actually does) #1338

Merged
merged 2 commits into from
Nov 12, 2019
Merged

Renamed more link to refresh (what it actually does) #1338

merged 2 commits into from
Nov 12, 2019

Conversation

eharris
Copy link
Contributor

@eharris eharris commented Oct 16, 2019

Also added Refresh SVG icons to match the text instead of reusing a not very appropriate css style.

Before:

Screen Shot 2019-10-16 at 00 11 57

After:

Screen Shot 2019-10-16 at 00 09 34

@eharris eharris mentioned this pull request Oct 16, 2019
Closed
@eharris eharris added in: UNKNOWN-frontend The problem location has not been identified yet. usability labels Oct 16, 2019
@eharris eharris force-pushed the rename-more-to-refresh branch from 61a65d0 to 460589f Compare October 17, 2019 21:05
@eharris
Copy link
Contributor Author

eharris commented Oct 17, 2019 •
edited
Loading

I realized I had a mismatched <td> block in the jsp, so fixed that.

@fxthomas fxthomas mentioned this pull request Nov 10, 2019
Merged
@muff1nman muff1nman merged commit 460589f into airsonic:master Nov 12, 2019
@eharris eharris deleted the rename-more-to-refresh branch November 12, 2019 07:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
1 more reviewer

@jvoisin jvoisin jvoisin approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
in: UNKNOWN-frontend The problem location has not been identified yet.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@eharris @jvoisin @muff1nman