feat(feedback): Align secondary buttons with the web - #4572

Merged
antonis merged 2 commits into
feedback-uifrom
antonis/feedback-button-design
Feb 20, 2025
Merged

feat(feedback): Align secondary buttons with the web#4572
antonis merged 2 commits into
feedback-uifrom
antonis/feedback-button-design

Conversation

@antonis

@antonisantonis commented Feb 20, 2025

Copy link
Copy Markdown
Contributor

📢 Type of change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring

📜 Description

Aligns secondary buttons with the web design

Web
web
iOSAndroid
afterScreenshot_1740043881

💡 Motivation and Context

Part of #4302

💚 How did you test it?

Manual

📝 Checklist

  • I added tests to verify changes
  • No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled
  • I updated the docs if needed.
  • I updated the wizard if needed.
  • All tests passing
  • No breaking changes

🔮 Next steps

#skip-changelog

@github-actions

github-actionsBot commented Feb 20, 2025

Copy link
Copy Markdown
Contributor
Messages
📖Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Generated by 🚫 dangerJS against 9ba38f5

@krystofwoldrichkrystofwoldrich left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good, thank you!

@github-actions

Copy link
Copy Markdown
Contributor

Android (legacy) Performance metrics 🚀

PlainWith SentryDiff
Startup time463.64 ms494.50 ms30.86 ms
Size17.75 MiB20.12 MiB2.38 MiB

Baseline results on branch: feedback-ui

Startup times

RevisionPlainWith SentryDiff
3e4cdf5462.35 ms474.96 ms12.61 ms
9402883448.53 ms468.73 ms20.20 ms
2646c98429.98 ms421.63 ms-8.35 ms
0325426477.32 ms457.43 ms-19.89 ms
bc11245435.30 ms421.73 ms-13.58 ms
e5d5735452.70 ms453.04 ms0.34 ms
269c976448.08 ms428.86 ms-19.22 ms
0459aee491.48 ms486.13 ms-5.35 ms
6b1624f462.78 ms465.13 ms2.35 ms
8cb898b438.83 ms420.58 ms-18.25 ms

App size

RevisionPlainWith SentryDiff
3e4cdf517.75 MiB20.12 MiB2.37 MiB
940288317.75 MiB20.12 MiB2.37 MiB
2646c9817.75 MiB20.12 MiB2.37 MiB
032542617.75 MiB20.12 MiB2.37 MiB
bc1124517.75 MiB20.12 MiB2.38 MiB
e5d573517.75 MiB20.12 MiB2.37 MiB
269c97617.75 MiB20.12 MiB2.37 MiB
0459aee17.75 MiB20.12 MiB2.37 MiB
6b1624f17.75 MiB20.12 MiB2.37 MiB
8cb898b17.75 MiB20.12 MiB2.37 MiB

@github-actions

Copy link
Copy Markdown
Contributor

iOS (legacy) Performance metrics 🚀

PlainWith SentryDiff
Startup time1233.43 ms1235.10 ms1.67 ms
Size2.63 MiB3.76 MiB1.12 MiB

Baseline results on branch: feedback-ui

Startup times

RevisionPlainWith SentryDiff
bc11245+dirty1220.78 ms1225.65 ms4.88 ms
2646c98+dirty1218.51 ms1218.92 ms0.41 ms
3e4cdf5+dirty1222.53 ms1224.42 ms1.89 ms
df05370+dirty1209.92 ms1216.55 ms6.63 ms
8cb898b+dirty1221.40 ms1231.78 ms10.37 ms
d1a10a1+dirty1223.68 ms1230.31 ms6.63 ms
b74349e+dirty1206.13 ms1221.65 ms15.53 ms
e5d5735+dirty1222.02 ms1222.22 ms0.20 ms
77e88fc+dirty1224.55 ms1232.41 ms7.86 ms
894ebb0+dirty1224.33 ms1214.45 ms-9.89 ms

App size

RevisionPlainWith SentryDiff
bc11245+dirty2.63 MiB3.76 MiB1.12 MiB
2646c98+dirty2.63 MiB3.69 MiB1.06 MiB
3e4cdf5+dirty2.63 MiB3.69 MiB1.06 MiB
df05370+dirty2.63 MiB3.71 MiB1.07 MiB
8cb898b+dirty2.63 MiB3.71 MiB1.08 MiB
d1a10a1+dirty2.63 MiB3.75 MiB1.12 MiB
b74349e+dirty2.63 MiB3.76 MiB1.12 MiB
e5d5735+dirty2.63 MiB3.69 MiB1.06 MiB
77e88fc+dirty2.63 MiB3.75 MiB1.12 MiB
894ebb0+dirty2.63 MiB3.71 MiB1.07 MiB

@github-actions

Copy link
Copy Markdown
Contributor

iOS (new) Performance metrics 🚀

PlainWith SentryDiff
Startup time1214.61 ms1210.19 ms-4.42 ms
Size3.19 MiB4.32 MiB1.14 MiB

Baseline results on branch: feedback-ui

Startup times

RevisionPlainWith SentryDiff
bc11245+dirty1229.67 ms1224.77 ms-4.90 ms
2646c98+dirty1239.94 ms1246.90 ms6.96 ms
3e4cdf5+dirty1213.36 ms1221.31 ms7.95 ms
df05370+dirty1221.14 ms1216.60 ms-4.55 ms
8cb898b+dirty1209.39 ms1207.57 ms-1.82 ms
d1a10a1+dirty1228.56 ms1229.79 ms1.23 ms
b74349e+dirty1214.22 ms1225.07 ms10.84 ms
e5d5735+dirty1217.78 ms1221.80 ms4.02 ms
77e88fc+dirty1218.79 ms1220.00 ms1.21 ms
894ebb0+dirty1210.94 ms1202.08 ms-8.85 ms

App size

RevisionPlainWith SentryDiff
bc11245+dirty3.19 MiB4.32 MiB1.14 MiB
2646c98+dirty3.19 MiB4.26 MiB1.07 MiB
3e4cdf5+dirty3.19 MiB4.26 MiB1.07 MiB
df05370+dirty3.19 MiB4.27 MiB1.09 MiB
8cb898b+dirty3.19 MiB4.28 MiB1.09 MiB
d1a10a1+dirty3.19 MiB4.32 MiB1.14 MiB
b74349e+dirty3.19 MiB4.32 MiB1.14 MiB
e5d5735+dirty3.19 MiB4.26 MiB1.07 MiB
77e88fc+dirty3.19 MiB4.32 MiB1.13 MiB
894ebb0+dirty3.19 MiB4.27 MiB1.09 MiB

@antonis
antonis merged commit ba260a4 into feedback-uiFeb 20, 2025
@antonis
antonis deleted the antonis/feedback-button-design branch February 20, 2025 10:44
Sign up for freeto 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

@antonis@krystofwoldrich
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Add copy buttons to all
 blocks\n(function() {\n function addCopyButtons() {\n document.querySelectorAll('pre code').forEach(function(codeBlock) {\n if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;\n codeBlock.parentElement.setAttribute('data-copy-added', 'true');\n \n var btn = document.createElement('button');\n btn.textContent = 'Copy';\n btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';\n btn.onmouseover = function() { this.style.opacity = '1'; };\n btn.onmouseout = function() { this.style.opacity = '0.7'; };\n btn.onclick = function() {\n navigator.clipboard.writeText(codeBlock.textContent).then(function() {\n btn.textContent = 'Copied!';\n setTimeout(function() { btn.textContent = 'Copy'; }, 1500);\n });\n };\n codeBlock.parentElement.style.position = 'relative';\n codeBlock.parentElement.appendChild(btn);\n });\n }\n \n addCopyButtons();\n \n // Re-run on dynamic content\n var observer = new MutationObserver(addCopyButtons);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Add Copy Buttons to Code Blocks");
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '
Skip to content

feat(feedback): Align secondary buttons with the web - #4572

Merged
antonis merged 2 commits into
feedback-uifrom
antonis/feedback-button-design
Feb 20, 2025
Merged

feat(feedback): Align secondary buttons with the web#4572
antonis merged 2 commits into
feedback-uifrom
antonis/feedback-button-design

Conversation

@antonis

@antonisantonis commented Feb 20, 2025

Copy link
Copy Markdown
Contributor

📢 Type of change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring

📜 Description

Aligns secondary buttons with the web design

Web
web
iOSAndroid
afterScreenshot_1740043881

💡 Motivation and Context

Part of #4302

💚 How did you test it?

Manual

📝 Checklist

  • I added tests to verify changes
  • No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled
  • I updated the docs if needed.
  • I updated the wizard if needed.
  • All tests passing
  • No breaking changes

🔮 Next steps

#skip-changelog

@github-actions

github-actionsBot commented Feb 20, 2025

Copy link
Copy Markdown
Contributor
Messages
📖Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Generated by 🚫 dangerJS against 9ba38f5

@krystofwoldrichkrystofwoldrich left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good, thank you!

@github-actions

Copy link
Copy Markdown
Contributor

Android (legacy) Performance metrics 🚀

PlainWith SentryDiff
Startup time463.64 ms494.50 ms30.86 ms
Size17.75 MiB20.12 MiB2.38 MiB

Baseline results on branch: feedback-ui

Startup times

RevisionPlainWith SentryDiff
3e4cdf5462.35 ms474.96 ms12.61 ms
9402883448.53 ms468.73 ms20.20 ms
2646c98429.98 ms421.63 ms-8.35 ms
0325426477.32 ms457.43 ms-19.89 ms
bc11245435.30 ms421.73 ms-13.58 ms
e5d5735452.70 ms453.04 ms0.34 ms
269c976448.08 ms428.86 ms-19.22 ms
0459aee491.48 ms486.13 ms-5.35 ms
6b1624f462.78 ms465.13 ms2.35 ms
8cb898b438.83 ms420.58 ms-18.25 ms

App size

RevisionPlainWith SentryDiff
3e4cdf517.75 MiB20.12 MiB2.37 MiB
940288317.75 MiB20.12 MiB2.37 MiB
2646c9817.75 MiB20.12 MiB2.37 MiB
032542617.75 MiB20.12 MiB2.37 MiB
bc1124517.75 MiB20.12 MiB2.38 MiB
e5d573517.75 MiB20.12 MiB2.37 MiB
269c97617.75 MiB20.12 MiB2.37 MiB
0459aee17.75 MiB20.12 MiB2.37 MiB
6b1624f17.75 MiB20.12 MiB2.37 MiB
8cb898b17.75 MiB20.12 MiB2.37 MiB

@github-actions

Copy link
Copy Markdown
Contributor

iOS (legacy) Performance metrics 🚀

PlainWith SentryDiff
Startup time1233.43 ms1235.10 ms1.67 ms
Size2.63 MiB3.76 MiB1.12 MiB

Baseline results on branch: feedback-ui

Startup times

RevisionPlainWith SentryDiff
bc11245+dirty1220.78 ms1225.65 ms4.88 ms
2646c98+dirty1218.51 ms1218.92 ms0.41 ms
3e4cdf5+dirty1222.53 ms1224.42 ms1.89 ms
df05370+dirty1209.92 ms1216.55 ms6.63 ms
8cb898b+dirty1221.40 ms1231.78 ms10.37 ms
d1a10a1+dirty1223.68 ms1230.31 ms6.63 ms
b74349e+dirty1206.13 ms1221.65 ms15.53 ms
e5d5735+dirty1222.02 ms1222.22 ms0.20 ms
77e88fc+dirty1224.55 ms1232.41 ms7.86 ms
894ebb0+dirty1224.33 ms1214.45 ms-9.89 ms

App size

RevisionPlainWith SentryDiff
bc11245+dirty2.63 MiB3.76 MiB1.12 MiB
2646c98+dirty2.63 MiB3.69 MiB1.06 MiB
3e4cdf5+dirty2.63 MiB3.69 MiB1.06 MiB
df05370+dirty2.63 MiB3.71 MiB1.07 MiB
8cb898b+dirty2.63 MiB3.71 MiB1.08 MiB
d1a10a1+dirty2.63 MiB3.75 MiB1.12 MiB
b74349e+dirty2.63 MiB3.76 MiB1.12 MiB
e5d5735+dirty2.63 MiB3.69 MiB1.06 MiB
77e88fc+dirty2.63 MiB3.75 MiB1.12 MiB
894ebb0+dirty2.63 MiB3.71 MiB1.07 MiB

@github-actions

Copy link
Copy Markdown
Contributor

iOS (new) Performance metrics 🚀

PlainWith SentryDiff
Startup time1214.61 ms1210.19 ms-4.42 ms
Size3.19 MiB4.32 MiB1.14 MiB

Baseline results on branch: feedback-ui

Startup times

RevisionPlainWith SentryDiff
bc11245+dirty1229.67 ms1224.77 ms-4.90 ms
2646c98+dirty1239.94 ms1246.90 ms6.96 ms
3e4cdf5+dirty1213.36 ms1221.31 ms7.95 ms
df05370+dirty1221.14 ms1216.60 ms-4.55 ms
8cb898b+dirty1209.39 ms1207.57 ms-1.82 ms
d1a10a1+dirty1228.56 ms1229.79 ms1.23 ms
b74349e+dirty1214.22 ms1225.07 ms10.84 ms
e5d5735+dirty1217.78 ms1221.80 ms4.02 ms
77e88fc+dirty1218.79 ms1220.00 ms1.21 ms
894ebb0+dirty1210.94 ms1202.08 ms-8.85 ms

App size

RevisionPlainWith SentryDiff
bc11245+dirty3.19 MiB4.32 MiB1.14 MiB
2646c98+dirty3.19 MiB4.26 MiB1.07 MiB
3e4cdf5+dirty3.19 MiB4.26 MiB1.07 MiB
df05370+dirty3.19 MiB4.27 MiB1.09 MiB
8cb898b+dirty3.19 MiB4.28 MiB1.09 MiB
d1a10a1+dirty3.19 MiB4.32 MiB1.14 MiB
b74349e+dirty3.19 MiB4.32 MiB1.14 MiB
e5d5735+dirty3.19 MiB4.26 MiB1.07 MiB
77e88fc+dirty3.19 MiB4.32 MiB1.13 MiB
894ebb0+dirty3.19 MiB4.27 MiB1.09 MiB

@antonis
antonis merged commit ba260a4 into feedback-uiFeb 20, 2025
@antonis
antonis deleted the antonis/feedback-button-design branch February 20, 2025 10:44
Sign up for freeto 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

@antonis@krystofwoldrich
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Force GitHub README to respect dark mode\n(function() {\n var style = document.createElement('style');\n style.textContent = '\n .markdown-body {\n color-scheme: dark light;\n }\n .markdown-body pre { background: #161b22 !important; }\n .markdown-body code { background: rgba(110, 118, 129, 0.4) !important; }\n .markdown-body table th, .markdown-body table td { border-color: #30363d !important; }\n .markdown-body img { background: #0d1117; }\n .markdown-body blockquote { border-left-color: #8b949e; }\n .markdown-body hr { border-color: #30363d; }\n ';\n document.head.appendChild(style);\n})();", "GitHub Dark Mode README Fix"); } } catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

feat(feedback): Align secondary buttons with the web - #4572

Merged
antonis merged 2 commits into
feedback-uifrom
antonis/feedback-button-design
Feb 20, 2025
Merged

feat(feedback): Align secondary buttons with the web#4572
antonis merged 2 commits into
feedback-uifrom
antonis/feedback-button-design

Conversation

@antonis

@antonisantonis commented Feb 20, 2025

Copy link
Copy Markdown
Contributor

📢 Type of change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring

📜 Description

Aligns secondary buttons with the web design

Web
web
iOSAndroid
afterScreenshot_1740043881

💡 Motivation and Context

Part of #4302

💚 How did you test it?

Manual

📝 Checklist

  • I added tests to verify changes
  • No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled
  • I updated the docs if needed.
  • I updated the wizard if needed.
  • All tests passing
  • No breaking changes

🔮 Next steps

#skip-changelog

@github-actions

github-actionsBot commented Feb 20, 2025

Copy link
Copy Markdown
Contributor
Messages
📖Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Generated by 🚫 dangerJS against 9ba38f5

@krystofwoldrichkrystofwoldrich left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good, thank you!

@github-actions

Copy link
Copy Markdown
Contributor

Android (legacy) Performance metrics 🚀

PlainWith SentryDiff
Startup time463.64 ms494.50 ms30.86 ms
Size17.75 MiB20.12 MiB2.38 MiB

Baseline results on branch: feedback-ui

Startup times

RevisionPlainWith SentryDiff
3e4cdf5462.35 ms474.96 ms12.61 ms
9402883448.53 ms468.73 ms20.20 ms
2646c98429.98 ms421.63 ms-8.35 ms
0325426477.32 ms457.43 ms-19.89 ms
bc11245435.30 ms421.73 ms-13.58 ms
e5d5735452.70 ms453.04 ms0.34 ms
269c976448.08 ms428.86 ms-19.22 ms
0459aee491.48 ms486.13 ms-5.35 ms
6b1624f462.78 ms465.13 ms2.35 ms
8cb898b438.83 ms420.58 ms-18.25 ms

App size

RevisionPlainWith SentryDiff
3e4cdf517.75 MiB20.12 MiB2.37 MiB
940288317.75 MiB20.12 MiB2.37 MiB
2646c9817.75 MiB20.12 MiB2.37 MiB
032542617.75 MiB20.12 MiB2.37 MiB
bc1124517.75 MiB20.12 MiB2.38 MiB
e5d573517.75 MiB20.12 MiB2.37 MiB
269c97617.75 MiB20.12 MiB2.37 MiB
0459aee17.75 MiB20.12 MiB2.37 MiB
6b1624f17.75 MiB20.12 MiB2.37 MiB
8cb898b17.75 MiB20.12 MiB2.37 MiB

@github-actions

Copy link
Copy Markdown
Contributor

iOS (legacy) Performance metrics 🚀

PlainWith SentryDiff
Startup time1233.43 ms1235.10 ms1.67 ms
Size2.63 MiB3.76 MiB1.12 MiB

Baseline results on branch: feedback-ui

Startup times

RevisionPlainWith SentryDiff
bc11245+dirty1220.78 ms1225.65 ms4.88 ms
2646c98+dirty1218.51 ms1218.92 ms0.41 ms
3e4cdf5+dirty1222.53 ms1224.42 ms1.89 ms
df05370+dirty1209.92 ms1216.55 ms6.63 ms
8cb898b+dirty1221.40 ms1231.78 ms10.37 ms
d1a10a1+dirty1223.68 ms1230.31 ms6.63 ms
b74349e+dirty1206.13 ms1221.65 ms15.53 ms
e5d5735+dirty1222.02 ms1222.22 ms0.20 ms
77e88fc+dirty1224.55 ms1232.41 ms7.86 ms
894ebb0+dirty1224.33 ms1214.45 ms-9.89 ms

App size

RevisionPlainWith SentryDiff
bc11245+dirty2.63 MiB3.76 MiB1.12 MiB
2646c98+dirty2.63 MiB3.69 MiB1.06 MiB
3e4cdf5+dirty2.63 MiB3.69 MiB1.06 MiB
df05370+dirty2.63 MiB3.71 MiB1.07 MiB
8cb898b+dirty2.63 MiB3.71 MiB1.08 MiB
d1a10a1+dirty2.63 MiB3.75 MiB1.12 MiB
b74349e+dirty2.63 MiB3.76 MiB1.12 MiB
e5d5735+dirty2.63 MiB3.69 MiB1.06 MiB
77e88fc+dirty2.63 MiB3.75 MiB1.12 MiB
894ebb0+dirty2.63 MiB3.71 MiB1.07 MiB

@github-actions

Copy link
Copy Markdown
Contributor

iOS (new) Performance metrics 🚀

PlainWith SentryDiff
Startup time1214.61 ms1210.19 ms-4.42 ms
Size3.19 MiB4.32 MiB1.14 MiB

Baseline results on branch: feedback-ui

Startup times

RevisionPlainWith SentryDiff
bc11245+dirty1229.67 ms1224.77 ms-4.90 ms
2646c98+dirty1239.94 ms1246.90 ms6.96 ms
3e4cdf5+dirty1213.36 ms1221.31 ms7.95 ms
df05370+dirty1221.14 ms1216.60 ms-4.55 ms
8cb898b+dirty1209.39 ms1207.57 ms-1.82 ms
d1a10a1+dirty1228.56 ms1229.79 ms1.23 ms
b74349e+dirty1214.22 ms1225.07 ms10.84 ms
e5d5735+dirty1217.78 ms1221.80 ms4.02 ms
77e88fc+dirty1218.79 ms1220.00 ms1.21 ms
894ebb0+dirty1210.94 ms1202.08 ms-8.85 ms

App size

RevisionPlainWith SentryDiff
bc11245+dirty3.19 MiB4.32 MiB1.14 MiB
2646c98+dirty3.19 MiB4.26 MiB1.07 MiB
3e4cdf5+dirty3.19 MiB4.26 MiB1.07 MiB
df05370+dirty3.19 MiB4.27 MiB1.09 MiB
8cb898b+dirty3.19 MiB4.28 MiB1.09 MiB
d1a10a1+dirty3.19 MiB4.32 MiB1.14 MiB
b74349e+dirty3.19 MiB4.32 MiB1.14 MiB
e5d5735+dirty3.19 MiB4.26 MiB1.07 MiB
77e88fc+dirty3.19 MiB4.32 MiB1.13 MiB
894ebb0+dirty3.19 MiB4.27 MiB1.09 MiB

@antonis
antonis merged commit ba260a4 into feedback-uiFeb 20, 2025
@antonis
antonis deleted the antonis/feedback-button-design branch February 20, 2025 10:44
Sign up for freeto 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

@antonis@krystofwoldrich
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Highlight search terms from Google/DuckDuckGo/Bing referrer\n(function() {\n var ref = document.referrer;\n var terms = [];\n \n if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) {\n var url = new URL(ref);\n var q = url.searchParams.get('q') || url.searchParams.get('p');\n if (q) {\n terms = q.split(/\\s+/).filter(function(t) { return t.length > 2; });\n }\n }\n \n if (terms.length === 0) return;\n \n var style = document.createElement('style');\n style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }';\n document.head.appendChild(style);\n \n function highlight(node) {\n if (node.nodeType === 3) { // text node\n var text = node.textContent;\n var found = false;\n terms.forEach(function(term) {\n var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\\]\\\\]/g, '\\\\') + ')', 'gi');\n if (regex.test(text)) {\n found = true;\n var frag = document.createDocumentFragment();\n var parts = text.split(regex);\n parts.forEach(function(part, i) {\n if (i % 2 === 0) {\n frag.appendChild(document.createTextNode(part));\n } else {\n var span = document.createElement('span');\n span.className = 'userscript-highlight';\n span.textContent = part;\n frag.appendChild(span);\n }\n });\n node.parentNode.replaceChild(frag, node);\n }\n });\n } else if (node.nodeType === 1 && node.childNodes) { // element\n var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT'];\n if (!skipTags.includes(node.tagName)) {\n Array.from(node.childNodes).forEach(highlight);\n }\n }\n }\n \n highlight(document.body);\n \n // Re-highlight on dynamic content\n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1 || node.nodeType === 3) highlight(node);\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Highlight Search Terms"); } } catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

feat(feedback): Align secondary buttons with the web - #4572

Merged
antonis merged 2 commits into
feedback-uifrom
antonis/feedback-button-design
Feb 20, 2025
Merged

feat(feedback): Align secondary buttons with the web#4572
antonis merged 2 commits into
feedback-uifrom
antonis/feedback-button-design

Conversation

@antonis

@antonisantonis commented Feb 20, 2025

Copy link
Copy Markdown
Contributor

📢 Type of change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring

📜 Description

Aligns secondary buttons with the web design

Web
web
iOSAndroid
afterScreenshot_1740043881

💡 Motivation and Context

Part of #4302

💚 How did you test it?

Manual

📝 Checklist

  • I added tests to verify changes
  • No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled
  • I updated the docs if needed.
  • I updated the wizard if needed.
  • All tests passing
  • No breaking changes

🔮 Next steps

#skip-changelog

@github-actions

github-actionsBot commented Feb 20, 2025

Copy link
Copy Markdown
Contributor
Messages
📖Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Generated by 🚫 dangerJS against 9ba38f5

@krystofwoldrichkrystofwoldrich left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good, thank you!

@github-actions

Copy link
Copy Markdown
Contributor

Android (legacy) Performance metrics 🚀

PlainWith SentryDiff
Startup time463.64 ms494.50 ms30.86 ms
Size17.75 MiB20.12 MiB2.38 MiB

Baseline results on branch: feedback-ui

Startup times

RevisionPlainWith SentryDiff
3e4cdf5462.35 ms474.96 ms12.61 ms
9402883448.53 ms468.73 ms20.20 ms
2646c98429.98 ms421.63 ms-8.35 ms
0325426477.32 ms457.43 ms-19.89 ms
bc11245435.30 ms421.73 ms-13.58 ms
e5d5735452.70 ms453.04 ms0.34 ms
269c976448.08 ms428.86 ms-19.22 ms
0459aee491.48 ms486.13 ms-5.35 ms
6b1624f462.78 ms465.13 ms2.35 ms
8cb898b438.83 ms420.58 ms-18.25 ms

App size

RevisionPlainWith SentryDiff
3e4cdf517.75 MiB20.12 MiB2.37 MiB
940288317.75 MiB20.12 MiB2.37 MiB
2646c9817.75 MiB20.12 MiB2.37 MiB
032542617.75 MiB20.12 MiB2.37 MiB
bc1124517.75 MiB20.12 MiB2.38 MiB
e5d573517.75 MiB20.12 MiB2.37 MiB
269c97617.75 MiB20.12 MiB2.37 MiB
0459aee17.75 MiB20.12 MiB2.37 MiB
6b1624f17.75 MiB20.12 MiB2.37 MiB
8cb898b17.75 MiB20.12 MiB2.37 MiB

@github-actions

Copy link
Copy Markdown
Contributor

iOS (legacy) Performance metrics 🚀

PlainWith SentryDiff
Startup time1233.43 ms1235.10 ms1.67 ms
Size2.63 MiB3.76 MiB1.12 MiB

Baseline results on branch: feedback-ui

Startup times

RevisionPlainWith SentryDiff
bc11245+dirty1220.78 ms1225.65 ms4.88 ms
2646c98+dirty1218.51 ms1218.92 ms0.41 ms
3e4cdf5+dirty1222.53 ms1224.42 ms1.89 ms
df05370+dirty1209.92 ms1216.55 ms6.63 ms
8cb898b+dirty1221.40 ms1231.78 ms10.37 ms
d1a10a1+dirty1223.68 ms1230.31 ms6.63 ms
b74349e+dirty1206.13 ms1221.65 ms15.53 ms
e5d5735+dirty1222.02 ms1222.22 ms0.20 ms
77e88fc+dirty1224.55 ms1232.41 ms7.86 ms
894ebb0+dirty1224.33 ms1214.45 ms-9.89 ms

App size

RevisionPlainWith SentryDiff
bc11245+dirty2.63 MiB3.76 MiB1.12 MiB
2646c98+dirty2.63 MiB3.69 MiB1.06 MiB
3e4cdf5+dirty2.63 MiB3.69 MiB1.06 MiB
df05370+dirty2.63 MiB3.71 MiB1.07 MiB
8cb898b+dirty2.63 MiB3.71 MiB1.08 MiB
d1a10a1+dirty2.63 MiB3.75 MiB1.12 MiB
b74349e+dirty2.63 MiB3.76 MiB1.12 MiB
e5d5735+dirty2.63 MiB3.69 MiB1.06 MiB
77e88fc+dirty2.63 MiB3.75 MiB1.12 MiB
894ebb0+dirty2.63 MiB3.71 MiB1.07 MiB

@github-actions

Copy link
Copy Markdown
Contributor

iOS (new) Performance metrics 🚀

PlainWith SentryDiff
Startup time1214.61 ms1210.19 ms-4.42 ms
Size3.19 MiB4.32 MiB1.14 MiB

Baseline results on branch: feedback-ui

Startup times

RevisionPlainWith SentryDiff
bc11245+dirty1229.67 ms1224.77 ms-4.90 ms
2646c98+dirty1239.94 ms1246.90 ms6.96 ms
3e4cdf5+dirty1213.36 ms1221.31 ms7.95 ms
df05370+dirty1221.14 ms1216.60 ms-4.55 ms
8cb898b+dirty1209.39 ms1207.57 ms-1.82 ms
d1a10a1+dirty1228.56 ms1229.79 ms1.23 ms
b74349e+dirty1214.22 ms1225.07 ms10.84 ms
e5d5735+dirty1217.78 ms1221.80 ms4.02 ms
77e88fc+dirty1218.79 ms1220.00 ms1.21 ms
894ebb0+dirty1210.94 ms1202.08 ms-8.85 ms

App size

RevisionPlainWith SentryDiff
bc11245+dirty3.19 MiB4.32 MiB1.14 MiB
2646c98+dirty3.19 MiB4.26 MiB1.07 MiB
3e4cdf5+dirty3.19 MiB4.26 MiB1.07 MiB
df05370+dirty3.19 MiB4.27 MiB1.09 MiB
8cb898b+dirty3.19 MiB4.28 MiB1.09 MiB
d1a10a1+dirty3.19 MiB4.32 MiB1.14 MiB
b74349e+dirty3.19 MiB4.32 MiB1.14 MiB
e5d5735+dirty3.19 MiB4.26 MiB1.07 MiB
77e88fc+dirty3.19 MiB4.32 MiB1.13 MiB
894ebb0+dirty3.19 MiB4.27 MiB1.09 MiB

@antonis
antonis merged commit ba260a4 into feedback-uiFeb 20, 2025
@antonis
antonis deleted the antonis/feedback-button-design branch February 20, 2025 10:44
Sign up for freeto 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

@antonis@krystofwoldrich
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Strip utm_, fbclid, gclid, etc. from all links on page\n(function() {\n var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',\n 'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',\n 'ref', 'ref_src', 'source', 'medium', 'campaign'];\n \n function cleanUrl(url) {\n try {\n var u = new URL(url, window.location.origin);\n var changed = false;\n trackingParams.forEach(function(p) {\n if (u.searchParams.has(p)) {\n u.searchParams.delete(p);\n changed = true;\n }\n });\n return changed ? u.toString() : url;\n } catch (e) {\n return url;\n }\n }\n \n function cleanLinks() {\n document.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n \n cleanLinks();\n \n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1) {\n if (node.tagName === 'A') cleanLinks();\n node.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Remove Tracking Parameters from Links"); } } catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); } })(); (function(){ try { var __m = "youtube.com"; var __re = new RegExp('^' + "youtube\\.com" + '
Skip to content

feat(feedback): Align secondary buttons with the web - #4572

Merged
antonis merged 2 commits into
feedback-uifrom
antonis/feedback-button-design
Feb 20, 2025
Merged

feat(feedback): Align secondary buttons with the web#4572
antonis merged 2 commits into
feedback-uifrom
antonis/feedback-button-design

Conversation

@antonis

@antonisantonis commented Feb 20, 2025

Copy link
Copy Markdown
Contributor

📢 Type of change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring

📜 Description

Aligns secondary buttons with the web design

Web
web
iOSAndroid
afterScreenshot_1740043881

💡 Motivation and Context

Part of #4302

💚 How did you test it?

Manual

📝 Checklist

  • I added tests to verify changes
  • No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled
  • I updated the docs if needed.
  • I updated the wizard if needed.
  • All tests passing
  • No breaking changes

🔮 Next steps

#skip-changelog

@github-actions

github-actionsBot commented Feb 20, 2025

Copy link
Copy Markdown
Contributor
Messages
📖Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Generated by 🚫 dangerJS against 9ba38f5

@krystofwoldrichkrystofwoldrich left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good, thank you!

@github-actions

Copy link
Copy Markdown
Contributor

Android (legacy) Performance metrics 🚀

PlainWith SentryDiff
Startup time463.64 ms494.50 ms30.86 ms
Size17.75 MiB20.12 MiB2.38 MiB

Baseline results on branch: feedback-ui

Startup times

RevisionPlainWith SentryDiff
3e4cdf5462.35 ms474.96 ms12.61 ms
9402883448.53 ms468.73 ms20.20 ms
2646c98429.98 ms421.63 ms-8.35 ms
0325426477.32 ms457.43 ms-19.89 ms
bc11245435.30 ms421.73 ms-13.58 ms
e5d5735452.70 ms453.04 ms0.34 ms
269c976448.08 ms428.86 ms-19.22 ms
0459aee491.48 ms486.13 ms-5.35 ms
6b1624f462.78 ms465.13 ms2.35 ms
8cb898b438.83 ms420.58 ms-18.25 ms

App size

RevisionPlainWith SentryDiff
3e4cdf517.75 MiB20.12 MiB2.37 MiB
940288317.75 MiB20.12 MiB2.37 MiB
2646c9817.75 MiB20.12 MiB2.37 MiB
032542617.75 MiB20.12 MiB2.37 MiB
bc1124517.75 MiB20.12 MiB2.38 MiB
e5d573517.75 MiB20.12 MiB2.37 MiB
269c97617.75 MiB20.12 MiB2.37 MiB
0459aee17.75 MiB20.12 MiB2.37 MiB
6b1624f17.75 MiB20.12 MiB2.37 MiB
8cb898b17.75 MiB20.12 MiB2.37 MiB

@github-actions

Copy link
Copy Markdown
Contributor

iOS (legacy) Performance metrics 🚀

PlainWith SentryDiff
Startup time1233.43 ms1235.10 ms1.67 ms
Size2.63 MiB3.76 MiB1.12 MiB

Baseline results on branch: feedback-ui

Startup times

RevisionPlainWith SentryDiff
bc11245+dirty1220.78 ms1225.65 ms4.88 ms
2646c98+dirty1218.51 ms1218.92 ms0.41 ms
3e4cdf5+dirty1222.53 ms1224.42 ms1.89 ms
df05370+dirty1209.92 ms1216.55 ms6.63 ms
8cb898b+dirty1221.40 ms1231.78 ms10.37 ms
d1a10a1+dirty1223.68 ms1230.31 ms6.63 ms
b74349e+dirty1206.13 ms1221.65 ms15.53 ms
e5d5735+dirty1222.02 ms1222.22 ms0.20 ms
77e88fc+dirty1224.55 ms1232.41 ms7.86 ms
894ebb0+dirty1224.33 ms1214.45 ms-9.89 ms

App size

RevisionPlainWith SentryDiff
bc11245+dirty2.63 MiB3.76 MiB1.12 MiB
2646c98+dirty2.63 MiB3.69 MiB1.06 MiB
3e4cdf5+dirty2.63 MiB3.69 MiB1.06 MiB
df05370+dirty2.63 MiB3.71 MiB1.07 MiB
8cb898b+dirty2.63 MiB3.71 MiB1.08 MiB
d1a10a1+dirty2.63 MiB3.75 MiB1.12 MiB
b74349e+dirty2.63 MiB3.76 MiB1.12 MiB
e5d5735+dirty2.63 MiB3.69 MiB1.06 MiB
77e88fc+dirty2.63 MiB3.75 MiB1.12 MiB
894ebb0+dirty2.63 MiB3.71 MiB1.07 MiB

@github-actions

Copy link
Copy Markdown
Contributor

iOS (new) Performance metrics 🚀

PlainWith SentryDiff
Startup time1214.61 ms1210.19 ms-4.42 ms
Size3.19 MiB4.32 MiB1.14 MiB

Baseline results on branch: feedback-ui

Startup times

RevisionPlainWith SentryDiff
bc11245+dirty1229.67 ms1224.77 ms-4.90 ms
2646c98+dirty1239.94 ms1246.90 ms6.96 ms
3e4cdf5+dirty1213.36 ms1221.31 ms7.95 ms
df05370+dirty1221.14 ms1216.60 ms-4.55 ms
8cb898b+dirty1209.39 ms1207.57 ms-1.82 ms
d1a10a1+dirty1228.56 ms1229.79 ms1.23 ms
b74349e+dirty1214.22 ms1225.07 ms10.84 ms
e5d5735+dirty1217.78 ms1221.80 ms4.02 ms
77e88fc+dirty1218.79 ms1220.00 ms1.21 ms
894ebb0+dirty1210.94 ms1202.08 ms-8.85 ms

App size

RevisionPlainWith SentryDiff
bc11245+dirty3.19 MiB4.32 MiB1.14 MiB
2646c98+dirty3.19 MiB4.26 MiB1.07 MiB
3e4cdf5+dirty3.19 MiB4.26 MiB1.07 MiB
df05370+dirty3.19 MiB4.27 MiB1.09 MiB
8cb898b+dirty3.19 MiB4.28 MiB1.09 MiB
d1a10a1+dirty3.19 MiB4.32 MiB1.14 MiB
b74349e+dirty3.19 MiB4.32 MiB1.14 MiB
e5d5735+dirty3.19 MiB4.26 MiB1.07 MiB
77e88fc+dirty3.19 MiB4.32 MiB1.13 MiB
894ebb0+dirty3.19 MiB4.27 MiB1.09 MiB

@antonis
antonis merged commit ba260a4 into feedback-uiFeb 20, 2025
@antonis
antonis deleted the antonis/feedback-button-design branch February 20, 2025 10:44
Sign up for freeto 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

@antonis@krystofwoldrich
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Auto-enable theater mode on YouTube\n(function() {\n function tryTheater() {\n var btn = document.querySelector('button[aria-label=\"Theater mode\"], ytd-player #player button[title=\"Theater mode\"]');\n if (btn && !btn.classList.contains('activated')) {\n btn.click();\n }\n }\n \n // Try immediately\n tryTheater();\n \n // Try after navigation (SPA)\n var lastUrl = location.href;\n setInterval(function() {\n if (location.href !== lastUrl) {\n lastUrl = location.href;\n setTimeout(tryTheater, 500);\n }\n }, 1000);\n \n // Also try on player load\n var observer = new MutationObserver(tryTheater);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "YouTube Theater Mode Default"); } } catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

feat(feedback): Align secondary buttons with the web - #4572

Merged
antonis merged 2 commits into
feedback-uifrom
antonis/feedback-button-design
Feb 20, 2025
Merged

feat(feedback): Align secondary buttons with the web#4572
antonis merged 2 commits into
feedback-uifrom
antonis/feedback-button-design

Conversation

@antonis

@antonisantonis commented Feb 20, 2025

Copy link
Copy Markdown
Contributor

📢 Type of change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring

📜 Description

Aligns secondary buttons with the web design

Web
web
iOSAndroid
afterScreenshot_1740043881

💡 Motivation and Context

Part of #4302

💚 How did you test it?

Manual

📝 Checklist

  • I added tests to verify changes
  • No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled
  • I updated the docs if needed.
  • I updated the wizard if needed.
  • All tests passing
  • No breaking changes

🔮 Next steps

#skip-changelog

@github-actions

github-actionsBot commented Feb 20, 2025

Copy link
Copy Markdown
Contributor
Messages
📖Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Generated by 🚫 dangerJS against 9ba38f5

@krystofwoldrichkrystofwoldrich left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good, thank you!

@github-actions

Copy link
Copy Markdown
Contributor

Android (legacy) Performance metrics 🚀

PlainWith SentryDiff
Startup time463.64 ms494.50 ms30.86 ms
Size17.75 MiB20.12 MiB2.38 MiB

Baseline results on branch: feedback-ui

Startup times

RevisionPlainWith SentryDiff
3e4cdf5462.35 ms474.96 ms12.61 ms
9402883448.53 ms468.73 ms20.20 ms
2646c98429.98 ms421.63 ms-8.35 ms
0325426477.32 ms457.43 ms-19.89 ms
bc11245435.30 ms421.73 ms-13.58 ms
e5d5735452.70 ms453.04 ms0.34 ms
269c976448.08 ms428.86 ms-19.22 ms
0459aee491.48 ms486.13 ms-5.35 ms
6b1624f462.78 ms465.13 ms2.35 ms
8cb898b438.83 ms420.58 ms-18.25 ms

App size

RevisionPlainWith SentryDiff
3e4cdf517.75 MiB20.12 MiB2.37 MiB
940288317.75 MiB20.12 MiB2.37 MiB
2646c9817.75 MiB20.12 MiB2.37 MiB
032542617.75 MiB20.12 MiB2.37 MiB
bc1124517.75 MiB20.12 MiB2.38 MiB
e5d573517.75 MiB20.12 MiB2.37 MiB
269c97617.75 MiB20.12 MiB2.37 MiB
0459aee17.75 MiB20.12 MiB2.37 MiB
6b1624f17.75 MiB20.12 MiB2.37 MiB
8cb898b17.75 MiB20.12 MiB2.37 MiB

@github-actions

Copy link
Copy Markdown
Contributor

iOS (legacy) Performance metrics 🚀

PlainWith SentryDiff
Startup time1233.43 ms1235.10 ms1.67 ms
Size2.63 MiB3.76 MiB1.12 MiB

Baseline results on branch: feedback-ui

Startup times

RevisionPlainWith SentryDiff
bc11245+dirty1220.78 ms1225.65 ms4.88 ms
2646c98+dirty1218.51 ms1218.92 ms0.41 ms
3e4cdf5+dirty1222.53 ms1224.42 ms1.89 ms
df05370+dirty1209.92 ms1216.55 ms6.63 ms
8cb898b+dirty1221.40 ms1231.78 ms10.37 ms
d1a10a1+dirty1223.68 ms1230.31 ms6.63 ms
b74349e+dirty1206.13 ms1221.65 ms15.53 ms
e5d5735+dirty1222.02 ms1222.22 ms0.20 ms
77e88fc+dirty1224.55 ms1232.41 ms7.86 ms
894ebb0+dirty1224.33 ms1214.45 ms-9.89 ms

App size

RevisionPlainWith SentryDiff
bc11245+dirty2.63 MiB3.76 MiB1.12 MiB
2646c98+dirty2.63 MiB3.69 MiB1.06 MiB
3e4cdf5+dirty2.63 MiB3.69 MiB1.06 MiB
df05370+dirty2.63 MiB3.71 MiB1.07 MiB
8cb898b+dirty2.63 MiB3.71 MiB1.08 MiB
d1a10a1+dirty2.63 MiB3.75 MiB1.12 MiB
b74349e+dirty2.63 MiB3.76 MiB1.12 MiB
e5d5735+dirty2.63 MiB3.69 MiB1.06 MiB
77e88fc+dirty2.63 MiB3.75 MiB1.12 MiB
894ebb0+dirty2.63 MiB3.71 MiB1.07 MiB

@github-actions

Copy link
Copy Markdown
Contributor

iOS (new) Performance metrics 🚀

PlainWith SentryDiff
Startup time1214.61 ms1210.19 ms-4.42 ms
Size3.19 MiB4.32 MiB1.14 MiB

Baseline results on branch: feedback-ui

Startup times

RevisionPlainWith SentryDiff
bc11245+dirty1229.67 ms1224.77 ms-4.90 ms
2646c98+dirty1239.94 ms1246.90 ms6.96 ms
3e4cdf5+dirty1213.36 ms1221.31 ms7.95 ms
df05370+dirty1221.14 ms1216.60 ms-4.55 ms
8cb898b+dirty1209.39 ms1207.57 ms-1.82 ms
d1a10a1+dirty1228.56 ms1229.79 ms1.23 ms
b74349e+dirty1214.22 ms1225.07 ms10.84 ms
e5d5735+dirty1217.78 ms1221.80 ms4.02 ms
77e88fc+dirty1218.79 ms1220.00 ms1.21 ms
894ebb0+dirty1210.94 ms1202.08 ms-8.85 ms

App size

RevisionPlainWith SentryDiff
bc11245+dirty3.19 MiB4.32 MiB1.14 MiB
2646c98+dirty3.19 MiB4.26 MiB1.07 MiB
3e4cdf5+dirty3.19 MiB4.26 MiB1.07 MiB
df05370+dirty3.19 MiB4.27 MiB1.09 MiB
8cb898b+dirty3.19 MiB4.28 MiB1.09 MiB
d1a10a1+dirty3.19 MiB4.32 MiB1.14 MiB
b74349e+dirty3.19 MiB4.32 MiB1.14 MiB
e5d5735+dirty3.19 MiB4.26 MiB1.07 MiB
77e88fc+dirty3.19 MiB4.32 MiB1.13 MiB
894ebb0+dirty3.19 MiB4.27 MiB1.09 MiB

@antonis
antonis merged commit ba260a4 into feedback-uiFeb 20, 2025
@antonis
antonis deleted the antonis/feedback-button-design branch February 20, 2025 10:44
Sign up for freeto 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

@antonis@krystofwoldrich
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Remove or un-stick sticky/fixed headers that block content\n(function() {\n function unstick() {\n document.querySelectorAll('header, nav, [role=\"banner\"], .header, .navbar, .sticky, .fixed-top, [style*=\"position: fixed\"], [style*=\"position:sticky\"]').forEach(function(el) {\n if (el.style.position === 'fixed' || el.style.position === 'sticky' || \n getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') {\n el.style.position = 'static';\n el.style.top = 'auto';\n el.style.zIndex = 'auto';\n }\n });\n }\n \n unstick();\n \n var observer = new MutationObserver(unstick);\n observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] });\n})();", "Kill Sticky Headers"); } } catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

feat(feedback): Align secondary buttons with the web - #4572

Merged
antonis merged 2 commits into
feedback-uifrom
antonis/feedback-button-design
Feb 20, 2025
Merged

feat(feedback): Align secondary buttons with the web#4572
antonis merged 2 commits into
feedback-uifrom
antonis/feedback-button-design

Conversation

@antonis

@antonisantonis commented Feb 20, 2025

Copy link
Copy Markdown
Contributor

📢 Type of change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring

📜 Description

Aligns secondary buttons with the web design

Web
web
iOSAndroid
afterScreenshot_1740043881

💡 Motivation and Context

Part of #4302

💚 How did you test it?

Manual

📝 Checklist

  • I added tests to verify changes
  • No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled
  • I updated the docs if needed.
  • I updated the wizard if needed.
  • All tests passing
  • No breaking changes

🔮 Next steps

#skip-changelog

@github-actions

github-actionsBot commented Feb 20, 2025

Copy link
Copy Markdown
Contributor
Messages
📖Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Generated by 🚫 dangerJS against 9ba38f5

@krystofwoldrichkrystofwoldrich left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good, thank you!

@github-actions

Copy link
Copy Markdown
Contributor

Android (legacy) Performance metrics 🚀

PlainWith SentryDiff
Startup time463.64 ms494.50 ms30.86 ms
Size17.75 MiB20.12 MiB2.38 MiB

Baseline results on branch: feedback-ui

Startup times

RevisionPlainWith SentryDiff
3e4cdf5462.35 ms474.96 ms12.61 ms
9402883448.53 ms468.73 ms20.20 ms
2646c98429.98 ms421.63 ms-8.35 ms
0325426477.32 ms457.43 ms-19.89 ms
bc11245435.30 ms421.73 ms-13.58 ms
e5d5735452.70 ms453.04 ms0.34 ms
269c976448.08 ms428.86 ms-19.22 ms
0459aee491.48 ms486.13 ms-5.35 ms
6b1624f462.78 ms465.13 ms2.35 ms
8cb898b438.83 ms420.58 ms-18.25 ms

App size

RevisionPlainWith SentryDiff
3e4cdf517.75 MiB20.12 MiB2.37 MiB
940288317.75 MiB20.12 MiB2.37 MiB
2646c9817.75 MiB20.12 MiB2.37 MiB
032542617.75 MiB20.12 MiB2.37 MiB
bc1124517.75 MiB20.12 MiB2.38 MiB
e5d573517.75 MiB20.12 MiB2.37 MiB
269c97617.75 MiB20.12 MiB2.37 MiB
0459aee17.75 MiB20.12 MiB2.37 MiB
6b1624f17.75 MiB20.12 MiB2.37 MiB
8cb898b17.75 MiB20.12 MiB2.37 MiB

@github-actions

Copy link
Copy Markdown
Contributor

iOS (legacy) Performance metrics 🚀

PlainWith SentryDiff
Startup time1233.43 ms1235.10 ms1.67 ms
Size2.63 MiB3.76 MiB1.12 MiB

Baseline results on branch: feedback-ui

Startup times

RevisionPlainWith SentryDiff
bc11245+dirty1220.78 ms1225.65 ms4.88 ms
2646c98+dirty1218.51 ms1218.92 ms0.41 ms
3e4cdf5+dirty1222.53 ms1224.42 ms1.89 ms
df05370+dirty1209.92 ms1216.55 ms6.63 ms
8cb898b+dirty1221.40 ms1231.78 ms10.37 ms
d1a10a1+dirty1223.68 ms1230.31 ms6.63 ms
b74349e+dirty1206.13 ms1221.65 ms15.53 ms
e5d5735+dirty1222.02 ms1222.22 ms0.20 ms
77e88fc+dirty1224.55 ms1232.41 ms7.86 ms
894ebb0+dirty1224.33 ms1214.45 ms-9.89 ms

App size

RevisionPlainWith SentryDiff
bc11245+dirty2.63 MiB3.76 MiB1.12 MiB
2646c98+dirty2.63 MiB3.69 MiB1.06 MiB
3e4cdf5+dirty2.63 MiB3.69 MiB1.06 MiB
df05370+dirty2.63 MiB3.71 MiB1.07 MiB
8cb898b+dirty2.63 MiB3.71 MiB1.08 MiB
d1a10a1+dirty2.63 MiB3.75 MiB1.12 MiB
b74349e+dirty2.63 MiB3.76 MiB1.12 MiB
e5d5735+dirty2.63 MiB3.69 MiB1.06 MiB
77e88fc+dirty2.63 MiB3.75 MiB1.12 MiB
894ebb0+dirty2.63 MiB3.71 MiB1.07 MiB

@github-actions

Copy link
Copy Markdown
Contributor

iOS (new) Performance metrics 🚀

PlainWith SentryDiff
Startup time1214.61 ms1210.19 ms-4.42 ms
Size3.19 MiB4.32 MiB1.14 MiB

Baseline results on branch: feedback-ui

Startup times

RevisionPlainWith SentryDiff
bc11245+dirty1229.67 ms1224.77 ms-4.90 ms
2646c98+dirty1239.94 ms1246.90 ms6.96 ms
3e4cdf5+dirty1213.36 ms1221.31 ms7.95 ms
df05370+dirty1221.14 ms1216.60 ms-4.55 ms
8cb898b+dirty1209.39 ms1207.57 ms-1.82 ms
d1a10a1+dirty1228.56 ms1229.79 ms1.23 ms
b74349e+dirty1214.22 ms1225.07 ms10.84 ms
e5d5735+dirty1217.78 ms1221.80 ms4.02 ms
77e88fc+dirty1218.79 ms1220.00 ms1.21 ms
894ebb0+dirty1210.94 ms1202.08 ms-8.85 ms

App size

RevisionPlainWith SentryDiff
bc11245+dirty3.19 MiB4.32 MiB1.14 MiB
2646c98+dirty3.19 MiB4.26 MiB1.07 MiB
3e4cdf5+dirty3.19 MiB4.26 MiB1.07 MiB
df05370+dirty3.19 MiB4.27 MiB1.09 MiB
8cb898b+dirty3.19 MiB4.28 MiB1.09 MiB
d1a10a1+dirty3.19 MiB4.32 MiB1.14 MiB
b74349e+dirty3.19 MiB4.32 MiB1.14 MiB
e5d5735+dirty3.19 MiB4.26 MiB1.07 MiB
77e88fc+dirty3.19 MiB4.32 MiB1.13 MiB
894ebb0+dirty3.19 MiB4.27 MiB1.09 MiB

@antonis
antonis merged commit ba260a4 into feedback-uiFeb 20, 2025
@antonis
antonis deleted the antonis/feedback-button-design branch February 20, 2025 10:44
Sign up for freeto 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

@antonis@krystofwoldrich
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Universal Dark Mode - works on any site\n(function() {\n var enabled = true;\n \n function applyDarkMode() {\n if (!enabled) return;\n \n // Create style element if it doesn't exist\n var style = document.getElementById('universal-dark-mode-style');\n if (!style) {\n style = document.createElement('style');\n style.id = 'universal-dark-mode-style';\n document.head.appendChild(style);\n }\n \n // Dark mode CSS - inverts colors but preserves images/video\n style.textContent = '\n /* Invert everything except media */\n html {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #1a1a2e !important;\n }\n \n /* Restore images, videos, iframes, canvas */\n img, video, iframe, canvas, svg, picture, [style*=\"background-image\"] {\n filter: invert(1) hue-rotate(180deg) !important;\n }\n \n /* Preserve specific elements that should not be inverted */\n .no-dark-mode, .no-dark-mode *,\n [data-theme=\"light\"], [data-theme=\"light\"],\n .ace_editor, .ace_editor *,\n .CodeMirror, .CodeMirror *,\n .monaco-editor, .monaco-editor *,\n .markdown-body pre, .markdown-body pre *,\n .highlight, .highlight *,\n pre code, pre code * {\n filter: none !important;\n }\n \n /* Fix common UI elements */\n .modal, .popup, .dropdown-menu, .tooltip, .popover {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #2d2d44 !important;\n border-color: #444 !important;\n }\n \n /* Scrollbars */\n ::-webkit-scrollbar { background: #1a1a2e !important; }\n ::-webkit-scrollbar-thumb { background: #444 !important; }\n ::-webkit-scrollbar-thumb:hover { background: #555 !important; }\n \n /* Selection */\n ::selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ';\n }\n \n function removeDarkMode() {\n var style = document.getElementById('universal-dark-mode-style');\n if (style) style.remove();\n }\n \n // Toggle with Alt+Shift+D\n document.addEventListener('keydown', function(e) {\n if (e.altKey && e.shiftKey && e.key === 'D') {\n e.preventDefault();\n enabled = !enabled;\n if (enabled) {\n applyDarkMode();\n console.log('[Universal Dark Mode] Enabled');\n } else {\n removeDarkMode();\n console.log('[Universal Dark Mode] Disabled');\n }\n }\n });\n \n // Apply on load\n applyDarkMode();\n \n // Re-apply on dynamic content\n var observer = new MutationObserver(function(mutations) {\n if (enabled && !document.getElementById('universal-dark-mode-style')) {\n applyDarkMode();\n }\n });\n observer.observe(document.head, { childList: true });\n \n console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle');\n})();", "Universal Dark Mode"); } } catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); } })(); })();
Skip to content

feat(feedback): Align secondary buttons with the web - #4572

Merged
antonis merged 2 commits into
feedback-uifrom
antonis/feedback-button-design
Feb 20, 2025
Merged

feat(feedback): Align secondary buttons with the web#4572
antonis merged 2 commits into
feedback-uifrom
antonis/feedback-button-design

Conversation

@antonis

@antonisantonis commented Feb 20, 2025

Copy link
Copy Markdown
Contributor

📢 Type of change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring

📜 Description

Aligns secondary buttons with the web design

Web
web
iOSAndroid
afterScreenshot_1740043881

💡 Motivation and Context

Part of #4302

💚 How did you test it?

Manual

📝 Checklist

  • I added tests to verify changes
  • No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled
  • I updated the docs if needed.
  • I updated the wizard if needed.
  • All tests passing
  • No breaking changes

🔮 Next steps

#skip-changelog

@github-actions

github-actionsBot commented Feb 20, 2025

Copy link
Copy Markdown
Contributor
Messages
📖Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Generated by 🚫 dangerJS against 9ba38f5

@krystofwoldrichkrystofwoldrich left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good, thank you!

@github-actions

Copy link
Copy Markdown
Contributor

Android (legacy) Performance metrics 🚀

PlainWith SentryDiff
Startup time463.64 ms494.50 ms30.86 ms
Size17.75 MiB20.12 MiB2.38 MiB

Baseline results on branch: feedback-ui

Startup times

RevisionPlainWith SentryDiff
3e4cdf5462.35 ms474.96 ms12.61 ms
9402883448.53 ms468.73 ms20.20 ms
2646c98429.98 ms421.63 ms-8.35 ms
0325426477.32 ms457.43 ms-19.89 ms
bc11245435.30 ms421.73 ms-13.58 ms
e5d5735452.70 ms453.04 ms0.34 ms
269c976448.08 ms428.86 ms-19.22 ms
0459aee491.48 ms486.13 ms-5.35 ms
6b1624f462.78 ms465.13 ms2.35 ms
8cb898b438.83 ms420.58 ms-18.25 ms

App size

RevisionPlainWith SentryDiff
3e4cdf517.75 MiB20.12 MiB2.37 MiB
940288317.75 MiB20.12 MiB2.37 MiB
2646c9817.75 MiB20.12 MiB2.37 MiB
032542617.75 MiB20.12 MiB2.37 MiB
bc1124517.75 MiB20.12 MiB2.38 MiB
e5d573517.75 MiB20.12 MiB2.37 MiB
269c97617.75 MiB20.12 MiB2.37 MiB
0459aee17.75 MiB20.12 MiB2.37 MiB
6b1624f17.75 MiB20.12 MiB2.37 MiB
8cb898b17.75 MiB20.12 MiB2.37 MiB

@github-actions

Copy link
Copy Markdown
Contributor

iOS (legacy) Performance metrics 🚀

PlainWith SentryDiff
Startup time1233.43 ms1235.10 ms1.67 ms
Size2.63 MiB3.76 MiB1.12 MiB

Baseline results on branch: feedback-ui

Startup times

RevisionPlainWith SentryDiff
bc11245+dirty1220.78 ms1225.65 ms4.88 ms
2646c98+dirty1218.51 ms1218.92 ms0.41 ms
3e4cdf5+dirty1222.53 ms1224.42 ms1.89 ms
df05370+dirty1209.92 ms1216.55 ms6.63 ms
8cb898b+dirty1221.40 ms1231.78 ms10.37 ms
d1a10a1+dirty1223.68 ms1230.31 ms6.63 ms
b74349e+dirty1206.13 ms1221.65 ms15.53 ms
e5d5735+dirty1222.02 ms1222.22 ms0.20 ms
77e88fc+dirty1224.55 ms1232.41 ms7.86 ms
894ebb0+dirty1224.33 ms1214.45 ms-9.89 ms

App size

RevisionPlainWith SentryDiff
bc11245+dirty2.63 MiB3.76 MiB1.12 MiB
2646c98+dirty2.63 MiB3.69 MiB1.06 MiB
3e4cdf5+dirty2.63 MiB3.69 MiB1.06 MiB
df05370+dirty2.63 MiB3.71 MiB1.07 MiB
8cb898b+dirty2.63 MiB3.71 MiB1.08 MiB
d1a10a1+dirty2.63 MiB3.75 MiB1.12 MiB
b74349e+dirty2.63 MiB3.76 MiB1.12 MiB
e5d5735+dirty2.63 MiB3.69 MiB1.06 MiB
77e88fc+dirty2.63 MiB3.75 MiB1.12 MiB
894ebb0+dirty2.63 MiB3.71 MiB1.07 MiB

@github-actions

Copy link
Copy Markdown
Contributor

iOS (new) Performance metrics 🚀

PlainWith SentryDiff
Startup time1214.61 ms1210.19 ms-4.42 ms
Size3.19 MiB4.32 MiB1.14 MiB

Baseline results on branch: feedback-ui

Startup times

RevisionPlainWith SentryDiff
bc11245+dirty1229.67 ms1224.77 ms-4.90 ms
2646c98+dirty1239.94 ms1246.90 ms6.96 ms
3e4cdf5+dirty1213.36 ms1221.31 ms7.95 ms
df05370+dirty1221.14 ms1216.60 ms-4.55 ms
8cb898b+dirty1209.39 ms1207.57 ms-1.82 ms
d1a10a1+dirty1228.56 ms1229.79 ms1.23 ms
b74349e+dirty1214.22 ms1225.07 ms10.84 ms
e5d5735+dirty1217.78 ms1221.80 ms4.02 ms
77e88fc+dirty1218.79 ms1220.00 ms1.21 ms
894ebb0+dirty1210.94 ms1202.08 ms-8.85 ms

App size

RevisionPlainWith SentryDiff
bc11245+dirty3.19 MiB4.32 MiB1.14 MiB
2646c98+dirty3.19 MiB4.26 MiB1.07 MiB
3e4cdf5+dirty3.19 MiB4.26 MiB1.07 MiB
df05370+dirty3.19 MiB4.27 MiB1.09 MiB
8cb898b+dirty3.19 MiB4.28 MiB1.09 MiB
d1a10a1+dirty3.19 MiB4.32 MiB1.14 MiB
b74349e+dirty3.19 MiB4.32 MiB1.14 MiB
e5d5735+dirty3.19 MiB4.26 MiB1.07 MiB
77e88fc+dirty3.19 MiB4.32 MiB1.13 MiB
894ebb0+dirty3.19 MiB4.27 MiB1.09 MiB

@antonis
antonis merged commit ba260a4 into feedback-uiFeb 20, 2025
@antonis
antonis deleted the antonis/feedback-button-design branch February 20, 2025 10:44
Sign up for freeto 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

@antonis@krystofwoldrich