GitLab

The data form for channel configuration looks like this:

  <command xmlns="http://jabber.org/protocol/commands" status="executing" node="configure" sessionid="1342523a-0c6a-484d-ba9e-1ca2b3075a8b">
    <x xmlns="jabber:x:data" type="form">
      <field type="list-single" label="Record history for this channel" var="record_history" desc="If unset, the value is the one configured globally">
        <value>true</value>
        <option label="unset">
          <value>unset</value>
        </option>
        <option label="true">
          <value>true</value>
        </option>
        <option label="false">
          <value>false</value>
        </option>
      </field>
      <field type="text-single" label="Out encoding" var="encoding_out" desc="The encoding used when sending messages to the IRC server. Defaults to the server's \xe2\x80\x9cout encoding\xe2\x80\x9d if unset for the channel">
        <value>ISO-8859-1</value>
      </field>
      <field type="text-single" label="In encoding" var="encoding_in" desc="The encoding used to decode message received from the IRC server. Defaults to the server's \xe2\x80\x9cin encoding\xe2\x80\x9d if unset for the channel">
        <value>ISO-8859-1</value>
      </field>
      <field type="boolean" label="Persistent" var="persistent" desc="If set to true, when all XMPP clients have left this channel, biboumi will stay idle in it, without sending a PART command.">
        <value>true</value>
      </field>
      <instructions>Edit the form, to configure the settings of the IRC channel #delphigl</instructions>
      <title>Configure the IRC channel #delphigl on server irc.euirc.net</title>
    </x>
    <actions>
      <next/>
    </actions>
  </command>

The options for the record_history variable could use more human-friendly names. I suggest:

• unset: "Server default (" + labels[the_actual_default] + ")"
• true (= labels[true]): Enabled
• false (= labels[false]): Disabled

The persistent field tooltip should talk about "enabled" and "disabled" instead of "true" and "false".

(I addressed the incorrect syntax for the field description in !31.)