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

Contributor update #2652

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from
Draft

Contributor update #2652

wants to merge 4 commits into from

Conversation

theofficialgman
Copy link
Collaborator

@theofficialgman theofficialgman commented Oct 3, 2024

@Botspot this is by no means final. I have created this PR so we can start to discuss the exact wording and the presentation of the information that we discussed in #dev-chat on the discord. The wording can (and likely should) fully change and was kept brief so as to not waste my time on something that we didn't agree on.

Comment on lines +545 to +549
yad "${yadflags[@]}" --class Pi-Apps --name "Pi-Apps" --width=700 --height=300 --title="Pi-Apps install/uninstall/updates complete" \
--image="${DIRECTORY}/icons/logo-64.png" --image-on-top \
--text='Thanks for using Pi-Apps!'$'\n'"The following apps were succcessful: $successful_apps" \
--wrap --fontname=12 \
--button='Ok'
Copy link
Collaborator Author

@theofficialgman theofficialgman Oct 3, 2024

Choose a reason for hiding this comment

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

This is the dialog I was suggesting that we have present the "thanks for using pi-apps" text with a call to action and the icons of the apps that were successfully installed/uninstalled/updated.

I think we had some code that presented all the app icons on one screen in the history of pi-apps somewhere. It would be interesting to do that but also show a small icon in the top left of each app icon for installing/uninstalling/updating.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I think we had some code that presented all the app icons on one screen in the history of pi-apps somewhere.

Yeah this:

d169e2a
7191be3
b7975c0

And I removed it as part of this because we were using the splashscreen to hide the fact that (at the time) pi-apps was slow to start:

7cd4f68

So we can bring that splashscreen code back (with modifications) for use in finish dialog.

Copy link
Collaborator Author

@theofficialgman theofficialgman Oct 4, 2024

Choose a reason for hiding this comment

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

So we can bring that splashscreen code back (with modifications) for use in finish dialog.

I brought it back b54a26d so we can more easily discuss how we would like to adjust it for use as described in the first comment in this thread.

@@ -420,6 +420,7 @@ $app"
[ -z "$geometry2" ] && geometry2='--center'

tail -f --retry "${DIRECTORY}/data/manage-daemon/yadlist" 2>/dev/null | yad --class Pi-Apps --name "Pi-Apps" --width=330 --height=400 "$geometry2" --title='Monitor Progress' \
--text="We are always looking for fresh ideas and new contributors to join our team: <a href="\""https://pi-apps.io/#get-involved"\"">https://pi-apps.io/#get-involved</a>" \
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

For this I want to make a dedicated webpage for "new contributors" to link to that goes over the things we discussed in the discord, attempting to welcome users into the "family" of pi-apps contributors.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant