Skip to content
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

gui: Update to Font Awesome v5 #4889

Merged
merged 4 commits into from May 24, 2018
Merged

gui: Update to Font Awesome v5 #4889

merged 4 commits into from May 24, 2018

Conversation

uok
Copy link
Contributor

@uok uok commented Apr 18, 2018

Update to Font Awesome v5 for more icons and bugfixes

  • update vendor/font-awesome
  • update HTML to new icon syntax
  • consistency of icon selection (e.g. all help buttons have help icon)
  • update vendor/fancytree
  • update vendor/daterangepicker

@uok
Copy link
Contributor Author

uok commented Apr 24, 2018

@calmh can you help with the two open points?

@calmh
Copy link
Member

calmh commented Apr 24, 2018

How so, what do you want me to do?

@wweich
Copy link
Member

wweich commented Apr 24, 2018

Regarding daterangepicker: I only see calendar and chevron icons used. The chevrons will work with FA5 as is, but the fa-calendar needs to be changed to fa-calendar-alt.
It seems, that there is not option to set the icon classes, so we have to patch it ourselves.

@AudriusButkevicius
Copy link
Member

What's the purpose of the upgrade, if we have to manually patch dependencies?

@uok uok requested a review from calmh as a code owner April 30, 2018 13:49
@uok
Copy link
Contributor Author

uok commented Apr 30, 2018

Update: daterangepicker has a new release and the script is no longer dependent on Bootstrap or Font Awesome

@uok
Copy link
Contributor Author

uok commented Apr 30, 2018

Great, now I wasted 1 hour of my life trying to merge my files with current master. 😠
This stupid software takes more time than the actual commit. 👎
Any help what to do at this point is greatly appreciated.

@calmh
Copy link
Member

calmh commented Apr 30, 2018

I think your PR might have been based on a quite old revision of master, as it looks like there are merge conflicts on unrelated Go files. If you can't sort it out by rebasing I'd create a new branch from (current) master and cherry-pick your change commits to it.

@uok
Copy link
Contributor Author

uok commented Apr 30, 2018

ok, PR good to go

</button>
<button type="button" class="btn btn-default btn-sm" ng-click="restoreVersions.show(folder.id)" ng-if="folder.versioning.type">
<span class="fa fa-undo"></span>&nbsp;<span translate>Versions</span>
<span class="fas fa-undo"></span>&nbsp;<span translate>Restore</span>
</button>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why the word change here?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In my opinion "Restore" is better suited because it describes what the button does and it is an action like "Pause", "Resume" and "Edit"

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I disagree.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This still has not been addressed.

@AudriusButkevicius
Copy link
Member

Conflicts and one comment.

@Catfriend1
Copy link
Contributor

Well the versions text should be in a separate PR as this is about the font update.

@uok
Copy link
Contributor Author

uok commented May 22, 2018

done!

@AudriusButkevicius
Copy link
Member

So I need to take it for a spin at some point I am by a computer which is probably tomorrow

@AudriusButkevicius AudriusButkevicius merged commit eb31be0 into syncthing:master May 24, 2018
@uok uok deleted the fontawesome-5 branch May 25, 2018 12:27
calmh added a commit to calmh/syncthing that referenced this pull request Jun 2, 2018
* master:
  gui: Disable rescan button while scanning (fixes syncthing#4977) (syncthing#4979)
  test: Add another variant of API timeout to skip when benchmarking
  gui, man: Update docs & translations
  build: Use commit date as assets change date
  script: Use source data from environment when generating assets
  lib/scanner: Copy execute bits from previous version on Windows (fixes syncthing#4969) (syncthing#4970)
  lib/watchaggregator: Speedup propagation of removals (fixes syncthing#4953)  (syncthing#4955)
  gui: Update to Font Awesome v5 (syncthing#4889)
  lib/watchaggregator: Prevent race on config update (syncthing#4938)
  lib/db: Update global count when removing the previous global version (syncthing#4968)
  gui: Check if folder exists in folderLabel (fixes syncthing#4965) (syncthing#4966)
  lib/model: Move String method to folder (syncthing#4964)
  gui, man: Update docs & translations
  lib/model: Refactor override implementation into sendOnlyFolder
  lib/model: Refactor folderScanner into folder
@calmh calmh added this to the v0.14.49 milestone Jun 5, 2018
@st-review st-review added the frozen-due-to-age Issues closed and untouched for a long time, together with being locked for discussion label May 25, 2019
@syncthing syncthing locked and limited conversation to collaborators May 25, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
frozen-due-to-age Issues closed and untouched for a long time, together with being locked for discussion
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants