Skip to content

Commit f8dbf3a

Browse files
sm-sayedism-sayedi
committed
icons: Add "person", "eye", and "eye_off" icons
Also renamed "user" to "two_person". New icons taken from: https://www.figma.com/design/1JTNtYo9memgW7vV6d0ygq/Zulip-Mobile?node-id=5968-237884&t=dku3J5Fv2dmWo7ht-0 https://www.figma.com/design/1JTNtYo9memgW7vV6d0ygq/Zulip-Mobile?node-id=5968-264632&t=dku3J5Fv2dmWo7ht-0
1 parent 8408098 commit f8dbf3a

File tree

12 files changed

+55
-37
lines changed

12 files changed

+55
-37
lines changed

‎assets/icons/ZulipIcons.ttf

816 Bytes
Binary file not shown.

‎assets/icons/eye.svg

Lines changed: 3 additions & 0 deletions
Loading

‎assets/icons/eye_off.svg

Lines changed: 3 additions & 0 deletions
Loading

‎assets/icons/person.svg

Lines changed: 3 additions & 0 deletions
Loading

‎assets/icons/user.svg renamed to ‎assets/icons/two_person.svg

File renamed without changes.

‎lib/widgets/home.dart

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -110,7 +110,7 @@ class _HomePageState extends State<HomePage> {
110110
narrow: const CombinedFeedNarrow()))),
111111
button(_HomePageTab.channels, ZulipIcons.hash_italic),
112112
// TODO(#1094): Users
113-
button(_HomePageTab.directMessages, ZulipIcons.user),
113+
button(_HomePageTab.directMessages, ZulipIcons.two_person),
114114
_NavigationBarButton( icon: ZulipIcons.menu,
115115
selected: false,
116116
onPressed: () => _showMainMenu(context, tabNotifier: _tab)),
@@ -514,7 +514,7 @@ class _DirectMessagesButton extends _NavigationBarMenuButton {
514514
const _DirectMessagesButton({required super.tabNotifier});
515515

516516
@override
517-
IconData get icon => ZulipIcons.user;
517+
IconData get icon => ZulipIcons.two_person;
518518

519519
@override
520520
String label(ZulipLocalizations zulipLocalizations) {

‎lib/widgets/icons.dart

Lines changed: 39 additions & 30 deletions
Original file line numberDiff line numberDiff line change
@@ -63,89 +63,98 @@ abstract final class ZulipIcons {
6363
/// The Zulip custom icon "copy".
6464
static const IconData copy = IconData(0xf10d, fontFamily: "Zulip Icons");
6565

66+
/// The Zulip custom icon "eye".
67+
static const IconData eye = IconData(0xf10e, fontFamily: "Zulip Icons");
68+
69+
/// The Zulip custom icon "eye_off".
70+
static const IconData eye_off = IconData(0xf10f, fontFamily: "Zulip Icons");
71+
6672
/// The Zulip custom icon "follow".
67-
static const IconData follow = IconData(0xf10e, fontFamily: "Zulip Icons");
73+
static const IconData follow = IconData(0xf110, fontFamily: "Zulip Icons");
6874

6975
/// The Zulip custom icon "format_quote".
70-
static const IconData format_quote = IconData(0xf10f, fontFamily: "Zulip Icons");
76+
static const IconData format_quote = IconData(0xf111, fontFamily: "Zulip Icons");
7177

7278
/// The Zulip custom icon "globe".
73-
static const IconData globe = IconData(0xf110, fontFamily: "Zulip Icons");
79+
static const IconData globe = IconData(0xf112, fontFamily: "Zulip Icons");
7480

7581
/// The Zulip custom icon "group_dm".
76-
static const IconData group_dm = IconData(0xf111, fontFamily: "Zulip Icons");
82+
static const IconData group_dm = IconData(0xf113, fontFamily: "Zulip Icons");
7783

7884
/// The Zulip custom icon "hash_italic".
79-
static const IconData hash_italic = IconData(0xf112, fontFamily: "Zulip Icons");
85+
static const IconData hash_italic = IconData(0xf114, fontFamily: "Zulip Icons");
8086

8187
/// The Zulip custom icon "hash_sign".
82-
static const IconData hash_sign = IconData(0xf113, fontFamily: "Zulip Icons");
88+
static const IconData hash_sign = IconData(0xf115, fontFamily: "Zulip Icons");
8389

8490
/// The Zulip custom icon "image".
85-
static const IconData image = IconData(0xf114, fontFamily: "Zulip Icons");
91+
static const IconData image = IconData(0xf116, fontFamily: "Zulip Icons");
8692

8793
/// The Zulip custom icon "inbox".
88-
static const IconData inbox = IconData(0xf115, fontFamily: "Zulip Icons");
94+
static const IconData inbox = IconData(0xf117, fontFamily: "Zulip Icons");
8995

9096
/// The Zulip custom icon "info".
91-
static const IconData info = IconData(0xf116, fontFamily: "Zulip Icons");
97+
static const IconData info = IconData(0xf118, fontFamily: "Zulip Icons");
9298

9399
/// The Zulip custom icon "inherit".
94-
static const IconData inherit = IconData(0xf117, fontFamily: "Zulip Icons");
100+
static const IconData inherit = IconData(0xf119, fontFamily: "Zulip Icons");
95101

96102
/// The Zulip custom icon "language".
97-
static const IconData language = IconData(0xf118, fontFamily: "Zulip Icons");
103+
static const IconData language = IconData(0xf11a, fontFamily: "Zulip Icons");
98104

99105
/// The Zulip custom icon "lock".
100-
static const IconData lock = IconData(0xf119, fontFamily: "Zulip Icons");
106+
static const IconData lock = IconData(0xf11b, fontFamily: "Zulip Icons");
101107

102108
/// The Zulip custom icon "menu".
103-
static const IconData menu = IconData(0xf11a, fontFamily: "Zulip Icons");
109+
static const IconData menu = IconData(0xf11c, fontFamily: "Zulip Icons");
104110

105111
/// The Zulip custom icon "message_checked".
106-
static const IconData message_checked = IconData(0xf11b, fontFamily: "Zulip Icons");
112+
static const IconData message_checked = IconData(0xf11d, fontFamily: "Zulip Icons");
107113

108114
/// The Zulip custom icon "message_feed".
109-
static const IconData message_feed = IconData(0xf11c, fontFamily: "Zulip Icons");
115+
static const IconData message_feed = IconData(0xf11e, fontFamily: "Zulip Icons");
110116

111117
/// The Zulip custom icon "mute".
112-
static const IconData mute = IconData(0xf11d, fontFamily: "Zulip Icons");
118+
static const IconData mute = IconData(0xf11f, fontFamily: "Zulip Icons");
119+
120+
/// The Zulip custom icon "person".
121+
static const IconData person = IconData(0xf120, fontFamily: "Zulip Icons");
113122

114123
/// The Zulip custom icon "read_receipts".
115-
static const IconData read_receipts = IconData(0xf11e, fontFamily: "Zulip Icons");
124+
static const IconData read_receipts = IconData(0xf121, fontFamily: "Zulip Icons");
116125

117126
/// The Zulip custom icon "send".
118-
static const IconData send = IconData(0xf11f, fontFamily: "Zulip Icons");
127+
static const IconData send = IconData(0xf122, fontFamily: "Zulip Icons");
119128

120129
/// The Zulip custom icon "settings".
121-
static const IconData settings = IconData(0xf120, fontFamily: "Zulip Icons");
130+
static const IconData settings = IconData(0xf123, fontFamily: "Zulip Icons");
122131

123132
/// The Zulip custom icon "share".
124-
static const IconData share = IconData(0xf121, fontFamily: "Zulip Icons");
133+
static const IconData share = IconData(0xf124, fontFamily: "Zulip Icons");
125134

126135
/// The Zulip custom icon "share_ios".
127-
static const IconData share_ios = IconData(0xf122, fontFamily: "Zulip Icons");
136+
static const IconData share_ios = IconData(0xf125, fontFamily: "Zulip Icons");
128137

129138
/// The Zulip custom icon "smile".
130-
static const IconData smile = IconData(0xf123, fontFamily: "Zulip Icons");
139+
static const IconData smile = IconData(0xf126, fontFamily: "Zulip Icons");
131140

132141
/// The Zulip custom icon "star".
133-
static const IconData star = IconData(0xf124, fontFamily: "Zulip Icons");
142+
static const IconData star = IconData(0xf127, fontFamily: "Zulip Icons");
134143

135144
/// The Zulip custom icon "star_filled".
136-
static const IconData star_filled = IconData(0xf125, fontFamily: "Zulip Icons");
145+
static const IconData star_filled = IconData(0xf128, fontFamily: "Zulip Icons");
137146

138147
/// The Zulip custom icon "three_person".
139-
static const IconData three_person = IconData(0xf126, fontFamily: "Zulip Icons");
148+
static const IconData three_person = IconData(0xf129, fontFamily: "Zulip Icons");
140149

141150
/// The Zulip custom icon "topic".
142-
static const IconData topic = IconData(0xf127, fontFamily: "Zulip Icons");
151+
static const IconData topic = IconData(0xf12a, fontFamily: "Zulip Icons");
143152

144-
/// The Zulip custom icon "unmute".
145-
static const IconData unmute = IconData(0xf128, fontFamily: "Zulip Icons");
153+
/// The Zulip custom icon "two_person".
154+
static const IconData two_person = IconData(0xf12b, fontFamily: "Zulip Icons");
146155

147-
/// The Zulip custom icon "user".
148-
static const IconData user = IconData(0xf129, fontFamily: "Zulip Icons");
156+
/// The Zulip custom icon "unmute".
157+
static const IconData unmute = IconData(0xf12c, fontFamily: "Zulip Icons");
149158

150159
// END GENERATED ICON DATA
151160
}

‎lib/widgets/inbox.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -319,7 +319,7 @@ class _AllDmsHeaderItem extends _HeaderItem {
319319

320320
@override String title(ZulipLocalizations zulipLocalizations) =>
321321
zulipLocalizations.recentDmConversationsSectionHeader;
322-
@override IconData get icon => ZulipIcons.user;
322+
@override IconData get icon => ZulipIcons.two_person;
323323

324324
// TODO(design) check if this is the right variable for these
325325
@override Color collapsedIconColor(context) => DesignVariables.of(context).labelMenuButton;

‎lib/widgets/message_list.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1207,7 +1207,7 @@ class DmRecipientHeader extends StatelessWidget {
12071207
child: Icon(
12081208
color: designVariables.title,
12091209
size: 16,
1210-
ZulipIcons.user)),
1210+
ZulipIcons.two_person)),
12111211
Expanded(
12121212
child: Text(title,
12131213
style: recipientHeaderTextStyle(context),

‎test/widgets/home_test.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -111,7 +111,7 @@ void main () {
111111
of: find.byType(ZulipAppBar),
112112
matching: find.text('Channels'))).findsOne();
113113

114-
await tester.tap(find.byIcon(ZulipIcons.user));
114+
await tester.tap(find.byIcon(ZulipIcons.two_person));
115115
await tester.pump();
116116
check(find.descendant(
117117
of: find.byType(ZulipAppBar),

‎test/widgets/message_list_test.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1174,7 +1174,7 @@ void main() {
11741174
final textSpan = tester.renderObject<RenderParagraph>(find.text(
11751175
zulipLocalizations.messageListGroupYouAndOthers(
11761176
zulipLocalizations.unknownUserName))).text;
1177-
final icon = tester.widget<Icon>(find.byIcon(ZulipIcons.user));
1177+
final icon = tester.widget<Icon>(find.byIcon(ZulipIcons.two_person));
11781178
check(textSpan).style.isNotNull().color.isNotNull().isSameColorAs(icon.color!);
11791179
});
11801180
});

‎test/widgets/recent_dm_conversations_test.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ Future<void> setupPage(WidgetTester tester, {
5858
// Switch to direct messages tab.
5959
await tester.tap(find.descendant(
6060
of: find.byType(Center),
61-
matching: find.byIcon(ZulipIcons.user)));
61+
matching: find.byIcon(ZulipIcons.two_person)));
6262
await tester.pump();
6363
}
6464

0 commit comments

Comments
 (0)