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

[AGM-1662] Update overview.md (iOS) #245

Merged
merged 6 commits into from
Jul 28, 2023
Merged

Conversation

vyanovsky
Copy link
Contributor

Some changes in accordance with AdGuard for iOS 4.5

Some changes in accordance with AdGuard for iOS 4.5
additional fixes
@vyanovsky vyanovsky requested a review from Versty July 24, 2023 08:34
@@ -95,7 +95,12 @@ Users can also handle their DNS security on the Network settings screen. *Filter

### DNS filtering {#dns-filtering}

To get access to this setting, *Advanced mode* has to be enabled from the *General settings* screen. After that, a new section will appear on the DNS protection screen. Tap it to customize your DNS traffic by applying own rules.
In DNS filtering you can customize your DNS traffic by enabling AdGuard DNS filter, adding custom DNS filters and using DNS
Copy link
Contributor

Choose a reason for hiding this comment

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

DNS filtering allows you to customize your DNS traffic by enabling the AdGuard DNS filter, adding custom DNS filters, and using the DNS block-/allowlist.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

blocklist/allowlist лучше полностью оставить думаю, т.к. специфичные термины, не для всех очевидна дихотомия

Copy link
Contributor Author

@vyanovsky vyanovsky Jul 27, 2023

Choose a reason for hiding this comment

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

и артикли the кажется немного перебор - тут типа такое общее перечисление что в целом можно делать (фичи), а так получается что "прям вот тот самый аг днс фильтр, и тот самый днс блоклист/аллоулист!!"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Итого на основе твоего я б вот так предложил:

DNS filtering allows you to customize your DNS traffic by enabling AdGuard DNS filter, adding custom DNS filters, and using DNS blocklist/allowlist.

Copy link
Contributor

Choose a reason for hiding this comment

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

AdGuard DNS filter пусть останется без артикля, потому что перед названиями фильтров их обычно не ставим, а перед чёрным и белым списком всё-таки нужен


How to access:

*Protection* 🛡 icon (the second icon in the bottom menu bar) → *DNS protection* → *DNS filtering*
Copy link
Contributor

Choose a reason for hiding this comment

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

не уверена, что использовать в кб эмодзи хорошая идея

Copy link
Contributor Author

@vyanovsky vyanovsky Jul 27, 2023

Choose a reason for hiding this comment

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

а почему нет? так просто нагляднее когда юзеру надо объяснить какую иконку тыкать, ведь на этом шаге в юае юзер не видит текст Protection

Copy link
Contributor Author

Choose a reason for hiding this comment

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

вот тут например юзаем эмодзи давно

https://adguard-vpn.com/kb/adguard-vpn-for-ios/overview/#app-settings

Copy link
Contributor

Choose a reason for hiding this comment

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

там эмодзи единичный, да и с его уместностью я бы поспорила

можно описать — the shield icon, что-то такое
база знаний больше похожа на документацию, это должен быть чистый текст без заигрываний с пользователем

Copy link
Contributor Author

@vyanovsky vyanovsky Jul 27, 2023

Choose a reason for hiding this comment

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

я бы не назвал это заигрыванием, тут эмодзи имеет чисто информативный характер, чтобы облегчить понимание юзеру какую иконку тыкать

все-таки это документация, призванная помочь юзеру понять как пользоваться / что делать

docs/adguard-for-ios/overview.md Outdated Show resolved Hide resolved
docs/adguard-for-ios/overview.md Outdated Show resolved Hide resolved
@@ -95,7 +95,12 @@ Users can also handle their DNS security on the Network settings screen. *Filter

### DNS filtering {#dns-filtering}

To get access to this setting, *Advanced mode* has to be enabled from the *General settings* screen. After that, a new section will appear on the DNS protection screen. Tap it to customize your DNS traffic by applying own rules.
In DNS filtering you can customize your DNS traffic by enabling AdGuard DNS filter, adding custom DNS filters and using DNS
Copy link
Contributor

Choose a reason for hiding this comment

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

AdGuard DNS filter пусть останется без артикля, потому что перед названиями фильтров их обычно не ставим, а перед чёрным и белым списком всё-таки нужен


How to access:

*Protection* 🛡 icon (the second icon in the bottom menu bar) → *DNS protection* → *DNS filtering*
Copy link
Contributor

Choose a reason for hiding this comment

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

там эмодзи единичный, да и с его уместностью я бы поспорила

можно описать — the shield icon, что-то такое
база знаний больше похожа на документацию, это должен быть чистый текст без заигрываний с пользователем

@vyanovsky vyanovsky merged commit c76d635 into master Jul 28, 2023
2 checks passed
@vyanovsky vyanovsky deleted the update-overview-md-ios branch July 28, 2023 08:22
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.

3 participants