Changes between Version 2 and Version 3 of DesignGuidelines

Timestamp:

Apr 26, 2007, 10:27:32 PM (17 years ago)

Author:

elb

Comment:

--

DesignGuidelines

@@ -9 +9 @@
 While Pidgin is a multiprotocol IM client, the goal is to hide protocols from the user as much as possible.  Obviously users have to know about individual protocols when they create or modify accounts, but in day-to-day communication and usage, the intent is that users don't have to think about protocols at all.  The workflow in Pidgin is intended to be "I would like to chat with Sean about wibbles", not "I would like to create an XMPP conversation with seanegan@pidgin.im".  The focus is on the goal, not the process.  In reaching toward this focus, we have chosen to paper over the differences between the various protocols and features as much as possible (without crippling or needlessly complicating things).  This has lead to decisions such as the removal of protocol icons from the buddy list and contact-aware chats and logs.
 
-== Core/UI/PrPL Split ==
+== Clean Layering ==
 
-In plain language, this means that the protocol-specific code goes in the protocol plugin (PrPl or prpl), and that libPurple exists.  There are practical implications to this.  While all of our code depends on glib, only the Pidgin specific parts depend on GTK+.  To implement, for example, file transfer, there are 3 steps.  First, the protocol(s) have to support it.  By themselves, however, the protocols can do nothing.  So the "core," libPurple, has to support it also (the second step).  We do not want massive amounts of very similar code in libPurple, so the implementation of file transfer at the libPurple level has to abstract away from how individual protocols handle it, so as to be able to use the same calls from all file transfer supporting protocols.  Last, but not least, before the user can actually send or receive a file, the UI (Pidgin, Finch or Adium) must support it.  These interfaces know nothing about the protocol, and have only limited contact with the core.  This helps to enforce the desire for uniformity explained above.  It also makes it easier for the only sort of duplication we encourage: many interfaces.  The core implementation cannot assume too much about what the UI will do, because the GTK+ UI (Pidgin) might need to handle a file transfer somewhat differently than the ncurses based UI (Finch).    Patches that break these splits will be rejected.  In practice, this means that there is more work involved to introduce a new class of functionality, say file transfer, white-boarding, voice, or video.  On the other hand, it means less work to implement any given class of functionality for a new protocol or for a new UI.  
+In plain language, this means that the protocol-specific code goes in the protocol plugin (PrPl or prpl), and that libPurple exists, and is cleanly separated from the user interface.  There are practical implications to this.  While all of our code depends on glib, only the Pidgin specific parts depend on GTK+.  To implement, for example, file transfer, there are 3 steps.  First, the protocol(s) have to support it.  By themselves, however, the protocols can do nothing.  So the "core," libPurple, has to support it also (the second step).  We do not want massive amounts of very similar code in libPurple, so the implementation of file transfer at the libPurple level has to abstract away from how individual protocols handle it, so as to be able to use the same calls from all file transfer supporting protocols.  Last, but not least, before the user can actually send or receive a file, the UI (Pidgin, Finch or Adium) must support it.  These interfaces know nothing about the protocol, and have only limited contact with the core.  This helps to enforce the desire for uniformity explained above.  It also makes it easier for the only sort of duplication we encourage: many interfaces.  The core implementation cannot assume too much about what the UI will do, because the GTK+ UI (Pidgin) might need to handle a file transfer somewhat differently than the ncurses based UI (Finch).  Patches that voilate this layering will be rejected.  In practice, this means that there is more work involved to introduce a new class of functionality, say file transfer, white-boarding, voice, or video.  On the other hand, it means less work to implement any given class of functionality for a new protocol or for a new UI.