Skip to content
This repository has been archived by the owner on Sep 9, 2024. It is now read-only.

Added a button to complete reading the wikipedia article via the Wike App #5

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

engalali9
Copy link
Contributor

@engalali9 engalali9 commented Apr 8, 2022

Screenshot from 2022-04-08 02-09-49

@nathandyer
Copy link
Collaborator

@ali-sameh-galal Thanks so much for this PR! This is almost ready to merge in, but I don't want to merge it until it it checks for Wike using Glib.AppInfo. Most folks that use Wike will have it installed from the flatpak (that's the primary way its distributed), and in that case /usr/bin/wike wouldn't exist. Plus, even for those that manually compile, it could be set to other paths depending on how you configure the build (installing to /usr/local/bin or ~/.local/bin is common).

@nathandyer nathandyer added enhancement New feature or request changes requested labels Apr 11, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
changes requested enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants