search

rpmfusion-infra / fedy

Fedy makes it easy to install third-party software in Fedora.
GNU General Public License v3.0
177 stars 20 forks source link

moved apps to terminal apps #47

Closed MalteKiefer closed 4 years ago

MalteKiefer commented 4 years ago

Signed-off-by: Malte Kiefer malte.kiefer@mailgermania.de

@kwizart moved apps to terminal apps

kwizart commented 4 years ago

I'm not sure about the interest of splitting apps between cmdline and graphical ones.

The main problem is the miss-categorization. If one is located in Apps, but end-users search it in Terminal Apps, it won't be found. Whereas with the search box one can use Apps for all and reduce the list by using a keyword. That way we could use as many Apps as possible, without changing the fedy application size. (which might fit well for smaller devices such as 12" screen, but not if extended with many categories)

All in all, I would prefer to denote the cmdline app by using a dedicate icon instead of a new category.

MalteKiefer commented 4 years ago

OK that is a good idea I think that is really the best way

04.02.2020 08:55:15 Nicolas Chauvet (kwizart) notifications@github.com:

I'm not sure about the interest of splitting apps between cmdline and graphical ones.

The main problem is the miss-categorization. If one is located in Apps, but end-users search it in Terminal Apps, it won't be found. Whereas with the search box one can use Apps for all and reduce the list by using a keyword. That way we could use as many Apps as possible, without changing the fedy application size. (which might fit well for smaller devices such as 12" screen, but not if extended with many categories)

All in all, I would prefer to denote the cmdline app by using a dedicate icon instead of a new category.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub [https://github.com/rpmfusion-infra/fedy/pull/47?email_source=notifications&email_token=AOD2H6KRANTWDUVGAO32RWDRBENODA5CNFSM4KPC2YD2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEKWVO4I#issuecomment-581785457] , or unsubscribe [https://github.com/notifications/unsubscribe-auth/AOD2H6JUW34ALVQIE57ORALRBENODANCNFSM4KPC2YDQ] . [https://github.com/notifications/beacon/AOD2H6J4WACSCSUPL5OVNFDRBENODA5CNFSM4KPC2YD2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEKWVO4I.gif]

kwizart commented 4 years ago

Seems like I forgot what we said in this PR (that should have been closed...) I will revert.