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

Adding About Info to Settings #103

Merged
merged 1 commit into from
Aug 7, 2024
Merged

Adding About Info to Settings #103

merged 1 commit into from
Aug 7, 2024

Conversation

Rahkeen
Copy link
Collaborator

@Rahkeen Rahkeen commented Aug 7, 2024

We discussed adding some attribution and links to an About section. I just added social links and a link to the emerge tools landing page.

I also added a placeholder for a feedback form, which we can fill in once we have a notion page or similar thing setup.

Lastly I wanted to add some fun + emerge-y branding so I created a card that has a sort of starfield animation. I built this a lil while ago and it felt fitting, can remove it if it's too extra.

We discussed adding some attribution and links to an About section. I just added social links and a link to the emerge tools landing page.

I also added a placeholder for a feedback form, which we can fill in once we have a notion page or similar thing setup.

Lastly I wanted to add some fun + emerge-y branding so I created a card that has a sort of starfield animation. I built this a lil while ago and it felt fitting, can remove it if it's too extra.
Copy link

emerge-tools bot commented Aug 7, 2024

2 builds increased size

Name Version Download Change Install Change Approval
⚠️ Hacker News
com.emergetools.hackernews
1.0 (1) 2.9 MB ⬆️ 374.0 kB (14.99%) 6.6 MB ⬆️ 800.2 kB (13.74%) N/A
⚠️ Hacker News
com.emergetools.hackernews.debug
1.0 (1) 12.7 MB ⬆️ 416.5 kB (3.39%) 36.3 MB ⬆️ 919.0 kB (2.6%) N/A

Hacker News 1.0 (1)
com.emergetools.hackernews

⚖️ Compare build
⏱️ Analyze build performance

Total install size change: ⬆️ 800.2 kB (13.74%)
Total download size change: ⬆️ 374.0 kB (14.99%)

Largest size changes

Item Install Size Change Download Size Change
📝 unbounded.ttf ⬆️ 384.0 kB ⬆️ 384.0 kB
📝 androidx.core.app.NavUtils ⬆️ 24.9 kB ⬆️ 11.3 kB
🗑 io.sentry.util.IntegrationUtils ⬇️ -18.9 kB ⬇️ -8.6 kB
🗑 io.sentry.util.LogUtils ⬇️ -16.6 kB ⬇️ -7.5 kB
🗑 io.sentry.util.HintUtils ⬇️ -14.9 kB ⬇️ -6.8 kB
View Treemap

Image of diff

Hacker News 1.0 (1)
com.emergetools.hackernews.debug

⚖️ Compare build
⏱️ Analyze build performance

Total install size change: ⬆️ 919.0 kB (2.6%)
Total download size change: ⬆️ 416.5 kB (3.39%)

Largest size changes

Item Install Size Change Download Size Change
📝 unbounded.ttf ⬆️ 384.0 kB ⬆️ 384.0 kB
📝 com.emergetools.hackernews.features.settings.SettingsComponentsKt ⬆️ 21.7 kB ⬆️ 8.3 kB
com.emergetools.hackernews.features.settings.SettingsScreenKt ⬇️ -10.0 kB ⬇️ -4.0 kB
📝 androidx.graphics.shapes.ShapesKt ⬆️ 7.8 kB ⬆️ 2.8 kB
📝 androidx.graphics.shapes.Cubic ⬆️ 5.0 kB ⬆️ 1.8 kB
View Treemap

Image of diff


🛸 Powered by Emerge Tools

Comment trigger: Size diff threshold of 100.00kB exceeded

Copy link

emerge-tools bot commented Aug 7, 2024

📸 Snapshot Test

2 modified, 5 added, 31 unchanged

Name Version Added Removed Modified Unchanged Errored Approval
Hacker News
com.emergetools.hackernews.debug
1.0 (1) 5 0 2 31 0 ✅ Approved

🛸 Powered by Emerge Tools

@Rahkeen Rahkeen merged commit 68e86e2 into main Aug 7, 2024
5 checks passed
@Rahkeen Rahkeen deleted the rikin.about-page branch August 7, 2024 22:03
Rahkeen added a commit that referenced this pull request Aug 8, 2024
We discussed adding some attribution and links to an About section. I
just added social links and a link to the emerge tools landing page.

I also added a placeholder for a feedback form, which we can fill in
once we have a notion page or similar thing setup.

Lastly I wanted to add some fun + emerge-y branding so I created a card
that has a sort of starfield animation. I built this a lil while ago and
it felt fitting, can remove it if it's too extra.
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.

2 participants